Skip to content

Bump deprecated GitHub Actions to current major versions#14315

Merged
giohappy merged 2 commits into
GeoNode:masterfrom
Valyrian-Code:ci/bump-deprecated-actions
Jun 23, 2026
Merged

Bump deprecated GitHub Actions to current major versions#14315
giohappy merged 2 commits into
GeoNode:masterfrom
Valyrian-Code:ci/bump-deprecated-actions

Conversation

@Valyrian-Code

Copy link
Copy Markdown
Contributor

Bumps GitHub Actions in the flake8 (Code formatting) and codeql (CodeQL) workflows from deprecated major versions to the current ones. No behaviour change.

  • actions/checkout@v3@v6 (matches the versions already used in tests.yml / run-test-suite.yml)
  • actions/setup-python@v2@v6
  • github/codeql-action/{init,autobuild,analyze}@v2@v3

Rationale: actions/checkout@v3 and actions/setup-python@v2 run on the end-of-life Node 16 runner, and github/codeql-action@v2 has been deprecated by GitHub. These emit deprecation warnings today and will eventually stop executing.

Checklist

For all pull requests:

  • Confirm you have read the contribution guidelines
  • You have sent a Contribution Licence Agreement (CLA) as necessary — CI-only change, no source or new files (small-change exemption)
  • Make sure the first PR targets the master branch

For core and extension modules:

  • There is a ticket describing the issue — CI maintenance, not visible to end users (template exemption for "changes not visible to end-users")
  • The issue connected to the PR must have Labels and Milestone assigned — N/A (no issue, per the exemption above)
  • PR for bug fixes and small new features are presented as a single commit
  • PR title must be in the form "[Fixes #<issue_number>] Title of the PR" — N/A (no issue, per the exemption above)
  • New unit tests have been added — N/A: this is a CI configuration change with no runtime code; correctness is exercised by this PR's own workflow runs

The flake8 and CodeQL workflows pinned actions to deprecated majors:
- actions/checkout@v3 and actions/setup-python@v2 run on the
  end-of-life Node 16 runtime.
- github/codeql-action@v2 (init/autobuild/analyze) is deprecated by GitHub.

Bump to current majors with no behaviour change:
- actions/checkout@v3  -> @v6 (matches tests.yml / run-test-suite.yml)
- actions/setup-python@v2 -> @v6
- github/codeql-action/*@v2 -> @V3
Copilot AI review requested due to automatic review settings June 9, 2026 19:53
@cla-bot

cla-bot Bot commented Jun 9, 2026

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @Valyrian-Code on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@Valyrian-Code

Copy link
Copy Markdown
Contributor Author

Signed CLA emailed to info@osgeo.org as @Valyrian-Code. Could a maintainer add me to the .clabot allowlist? This is a small CI-only change. Thanks!

@Valyrian-Code

Copy link
Copy Markdown
Contributor Author

Hi @mattiagiupponi, friendly follow-up on this. I emailed the signed CLA to info@osgeo.org as @Valyrian-Code on June 9. Could you add me to the .clabot allowlist so this can pass? It is a small CI-only change (bumping deprecated GitHub Actions). Thanks!

@Valyrian-Code

Copy link
Copy Markdown
Contributor Author

Hi @mattiagiupponi, friendly follow-up on this. I emailed the signed CLA to info@osgeo.org as @Valyrian-Code on June 9. Could you add me to the .clabot allowlist so this can pass? It is a small CI-only change (bumping deprecated GitHub Actions). Thanks!

@giohappy

Copy link
Copy Markdown
Contributor

@Valyrian-Code @RAJVEER42 your contributions look good and we thank you for them, but they're clearly (partially if not fully) generated by an LLM driven bot.
We cannot let a bot account sign a CLA, for all the ethical and legal implications.

We invite you either to sign and submit the PRs as the legally responsible owner/operator of the bot.
Otheriwse we will not be able to accept the PRs directly. You can continue sending them but we will close and re-create them case by case with our own accounts.

@Valyrian-Code

Copy link
Copy Markdown
Contributor Author

@giohappy

Thanks for the feedback. Just to clarify, I'm a real person (Rajveer Bishnoi), not a bot. I do use AI tools to assist my workflow, but I personally review, understand, test, and submit every change myself, and I'm fully responsible for the code I contribute.

Sorry if my activity gave a different impression. I'm happy to explain the reasoning behind this change or make any adjustments needed.

I'm interested in contributing to GeoNode. Thanks again for taking the time to review the PR.

GIF

@giohappy

Copy link
Copy Markdown
Contributor

@Valyrian-Code you can add your account to the clabot file inside this PR. We will review it.

@cla-bot

cla-bot Bot commented Jun 22, 2026

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @Valyrian-Code on file. In order for us to review and merge your code, please contact the project maintainers to get yourself added.

@Valyrian-Code

Valyrian-Code commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Done, added Valyrian-Code to the .clabot file in this PR (commit a2a7dda). Thanks for the guidance, @giohappy. Happy to adjust anything else.

@giohappy giohappy self-requested a review June 23, 2026 07:42
@giohappy giohappy merged commit 68d1b41 into GeoNode:master Jun 23, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants