Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mark .form files as inputs to :instrumentForms task #4685

Conversation

kwvanderlinde
Copy link
Collaborator

@kwvanderlinde kwvanderlinde commented Feb 8, 2024

Identify the Bug or Feature request

Improves on PR #4390 for #4389

Description of the Change

I've noticed that the current :instrumentForms task does not recognize changes to IntelliJ GUI files. This makes dev iteration more clunky than it needs to be when making frequent UI changes. So the situation was only mildly improved by the previous PR.

This PR makes it so the class files are reinstrumented whenever a .form file is changed. This way they won't report as up-to-date unless neither Java nor form files are modified.

Possible Drawbacks

Should be none (build only).

Documentation Notes

N/A

Release Notes

N/A


This change is Reviewable

This makes the class files to be reinstrumented when GUI designer changes are made. Previously tasks would all report as
UP-TO-DATE, so changes would not be reflected in the next run.
@kwvanderlinde
Copy link
Collaborator Author

This is a part of Gradle I'm not super familiar with. If anyone knows a more idiomatic way to grab the .form files as task inputs, I'll update accordingly.

@cwisniew cwisniew added this pull request to the merge queue Feb 8, 2024
@cwisniew cwisniew added the code-maintenance Adding/editing javadocs, unit tests, formatting. label Feb 8, 2024
Merged via the queue into RPTools:develop with commit 0100652 Feb 8, 2024
4 checks passed
@kwvanderlinde kwvanderlinde deleted the refactor/4389-reinstrument-on-form-file-changes branch February 8, 2024 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-maintenance Adding/editing javadocs, unit tests, formatting.
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants