Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # 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.

3 participants