You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@creadur.apache.org by "Claudenw (via GitHub)" <gi...@apache.org> on 2023/10/07 14:09:36 UTC
[PR] Rat 321 text based configuration [creadur-rat]
Claudenw opened a new pull request, #157:
URL: https://github.com/apache/creadur-rat/pull/157
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "Claudenw (via GitHub)" <gi...@apache.org>.
Claudenw commented on PR #157:
URL: https://github.com/apache/creadur-rat/pull/157#issuecomment-1764200268
There are differences in the UI for Maven and Ant when not using the
standard definitions.
Preparing a release and collecting feedback seems like a viable path. I
think we need to update documentation, but I think a release candidate is a
good start.
Claude
On Mon, Oct 16, 2023 at 11:29 AM P. Ottlinger ***@***.***>
wrote:
> @Claudenw <https://github.com/Claudenw> if I get it correctly there are
> no functional changes of existing functionality. Thus I'd prefer to merge
> your changes and prepare a release of RAT in order to collect feedback from
> existing users. WDYT?
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/creadur-rat/pull/157#issuecomment-1764081462>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AASTVHSEHJGAOTVR77ASA6TX7T5AXAVCNFSM6AAAAAA5W6RTVWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRUGA4DCNBWGI>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
LinkedIn: http://www.linkedin.com/in/claudewarren
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "Claudenw (via GitHub)" <gi...@apache.org>.
Claudenw commented on PR #157:
URL: https://github.com/apache/creadur-rat/pull/157#issuecomment-1768516412
@ottlinger how do we proceed? I do not have access to merge and I am unsure of the process to create a release so I think I need you to guide and/or execute this process. If you need anything from me please let me know.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "Claudenw (via GitHub)" <gi...@apache.org>.
Claudenw commented on PR #157:
URL: https://github.com/apache/creadur-rat/pull/157#issuecomment-1752108298
I recognize that this is a massive change and will take some time to process. If you have any questions or want to see extra tests please let me know and I will endeavour to complete them as quickly as possible.
It may make sense to create a new branch to put these changes on so that we can get an alpha release out to get some feedback before going all in on the changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "ottlinger (via GitHub)" <gi...@apache.org>.
ottlinger commented on PR #157:
URL: https://github.com/apache/creadur-rat/pull/157#issuecomment-1769177998
@Claudenw I'll start preparations for a new RC during the weekend (as the changelog and Jira needs some more attention before a new RC can be created). How would you summarize the feature RAT-321 introduces (feel free to add this to this branch's src/main/changes.xml))
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "ottlinger (via GitHub)" <gi...@apache.org>.
ottlinger merged PR #157:
URL: https://github.com/apache/creadur-rat/pull/157
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] RAT-321: text based configuration [creadur-rat]
Posted by "ottlinger (via GitHub)" <gi...@apache.org>.
ottlinger commented on PR #157:
URL: https://github.com/apache/creadur-rat/pull/157#issuecomment-1764081462
@Claudenw if I get it correctly there are no functional changes of existing functionality. Thus I'd prefer to merge your changes and prepare a release of RAT in order to collect feedback from existing users. WDYT?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@creadur.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org