Skip to content

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

Merged
cwisniew merged 1 commit into
RPTools:developfrom
kwvanderlinde:refactor/4389-reinstrument-on-form-file-changes
Feb 8, 2024
Merged

Mark .form files as inputs to :instrumentForms task#4685
cwisniew merged 1 commit into
RPTools:developfrom
kwvanderlinde:refactor/4389-reinstrument-on-form-file-changes

Conversation

@kwvanderlinde

@kwvanderlinde kwvanderlinde commented Feb 8, 2024

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
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
@kwvanderlinde
kwvanderlinde deleted the refactor/4389-reinstrument-on-form-file-changes branch February 8, 2024 23:53
@kwvanderlinde kwvanderlinde linked an issue Feb 13, 2025 that may be closed by this pull request
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

No open projects
Status: Merged

Development

Successfully merging this pull request may close these issues.

[Refactoring]: Avoid deleting .class files every build

2 participants