Skip to content

update update scripts#769

Merged
boomzero merged 2 commits intodevfrom
boomzero/update-scripts
Feb 8, 2025
Merged

update update scripts#769
boomzero merged 2 commits intodevfrom
boomzero/update-scripts

Conversation

@boomzero
Copy link
Member

@boomzero boomzero commented Feb 8, 2025

What does this PR aim to accomplish?:

This pull request includes updates to the Update scripts to improve version management and handling of specific markers in JSON files. The most important changes include updating the npm version command and handling the //!ci-no-touch marker.

How does this PR accomplish the above?:

Version management improvements:

Handling of //!ci-no-touch marker:

  • Update/UpdateVersion.js: Added logic to remove the //!ci-no-touch marker, reconfigure git settings, pull the latest changes, commit the removal, and push the changes.

By submitting this pull request, I confirm the following:

  1. I have read and understood the contributor's guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented on my proposed changes within the code and I have tested my changes.
  3. I am willing to help maintain this change if there are issues with it later.
  4. It is compatible with the GNU General Public License v3.0
  5. I have squashed any insignificant commits. (git rebase)
  6. I have checked that another pull request for this purpose does not exist.
  7. I have considered and confirmed that this submission will be valuable to others.
  8. I accept that this submission may not be used, and the pull request can be closed at the will of the maintainer.
  9. I give this submission freely and claim no ownership to its content.

  • I have read the above and my PR is ready for review. Check this box to confirm

@hendragon-bot hendragon-bot bot added the update-script Related to our update infrastructure! label Feb 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2025

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@boomzero boomzero merged commit 724e51a into dev Feb 8, 2025
9 checks passed
@boomzero boomzero deleted the boomzero/update-scripts branch February 8, 2025 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS update-script Related to our update infrastructure!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant