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

Use relative paths in create_contributor_list.py #7656

Merged

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Oct 21, 2022

Description

On MacOS Path(__file__).parent.parent seems to be resolved to an absolute path. This causes the contibutors-txt script to insert a duplicate header as the path for .contributors_aliases.json doesn't match anymore.
Noticed this while testing the release workflow.

https://github.com/PyCQA/pylint/blob/15dd079792ce8e6b37856450f9e4a928f11b3549/CONTRIBUTORS.txt#L1-L5

Refs: #7362

@cdce8p cdce8p added Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer Maintenance Discussion or action around maintaining pylint or the dev workflow labels Oct 21, 2022
@cdce8p cdce8p added this to the 2.15.5 milestone Oct 21, 2022
@cdce8p cdce8p added the Skip news 🔇 This change does not require a changelog entry label Oct 21, 2022
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3296438008

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.349%

Totals Coverage Status
Change from base Build 3290498180: 0.0%
Covered Lines: 17138
Relevant Lines: 17974

💛 - Coveralls

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.

Thank you ! This is part of #7362 please do not hesitate to edit the description or comment on it :)

@Pierre-Sassoulas Pierre-Sassoulas merged commit 09dbd29 into pylint-dev:main Oct 21, 2022
@cdce8p cdce8p deleted the contributors-relative-paths branch October 21, 2022 11:25
cdce8p added a commit to cdce8p/pylint that referenced this pull request Oct 21, 2022
@cdce8p cdce8p added Backported and removed Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer labels Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backported 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.

None yet

3 participants