Skip to content

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Sep 3, 2025

Description

Followup to #10396

I finally got around to forking and updating the backport action. I can be found here
https://github.com/pylint-dev/backport
tibdex/backport@main...pylint-dev:backport:main

The plan is to due only the bare minimum of maintenance there. So far I've

  • Updated node to v24 and the dependencies
  • Fixed some workflows
  • Added user_name and user_email options

I've configured the vars in the pylint-dev org selected for the pylint and astroid repos.

Open question

Should we rename the pylint-backport-bot[bot] to just pylint-backport[bot]? When I did set it up, I didn't know that Github would add [bot] automatically. It's probably just a style thing. At most some commits in the maintenance branch won't be probably attributed to the bot anymore.

I lean slightly towards doing it.

--
Will tag this one for the backport, just so we can see if everything worked out fine.

@cdce8p cdce8p added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news 🔇 This change does not require a changelog entry backport maintenance/3.3.x labels Sep 3, 2025
Copy link

codecov bot commented Sep 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.84%. Comparing base (2d60c59) to head (6c6d2a4).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #10536      +/-   ##
==========================================
- Coverage   95.84%   95.84%   -0.01%     
==========================================
  Files         177      177              
  Lines       19282    19278       -4     
==========================================
- Hits        18481    18477       -4     
  Misses        801      801              

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

github-actions bot commented Sep 3, 2025

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit 6c6d2a4

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

Great ! Let's not move too much from the main branch in case we want to rebase on it or go back to it maybe ?

@cdce8p
Copy link
Member Author

cdce8p commented Sep 3, 2025

Let's not move too much from the main branch in case we want to rebase on it or go back to it maybe ?

I agree. My main focus here was to a) bring it in a maintainable state / fix all security warnings and b) implement the user name and email inputs. I don't plan to do any more changes at the moment. https://github.com/pylint-dev/backport/security/dependabot?q=is%3Aclosed

Tbh though I don't know if the upstream repo is even still maintained anymore. There hasn't been any activity for over two years. https://github.com/tibdex/backport

Should we rename the pylint-backport-bot[bot] to just pylint-backport[bot]?

Just did that, looks like it worked out fine. All past commits are still attributed correctly.

@cdce8p cdce8p merged commit 6f8da73 into pylint-dev:main Sep 3, 2025
40 checks passed
@cdce8p cdce8p deleted the update-backport-workflow branch September 3, 2025 20:30
pylint-backport bot pushed a commit that referenced this pull request Sep 3, 2025
(cherry picked from commit 6f8da73)
cdce8p added a commit that referenced this pull request Sep 3, 2025
(cherry picked from commit 6f8da73)

Co-authored-by: Marc Mueller <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport maintenance/3.3.x Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news 🔇 This change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants