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 #1845

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.

https://github.com/PyCQA/astroid/blob/1a074cf66fc4fdb6714fac7afb14f9e7457cd704/CONTRIBUTORS.txt#L1-L5

Refs

@cdce8p cdce8p added the Maintenance Discussion or action around maintaining astroid or the dev workflow label Oct 21, 2022
@coveralls
Copy link

Pull Request Test Coverage Report for Build 3297107319

  • 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 92.272%

Totals Coverage Status
Change from base Build 3284753465: 0.0%
Covered Lines: 9862
Relevant Lines: 10688

💛 - 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.

👌

@Pierre-Sassoulas Pierre-Sassoulas merged commit 3329d23 into pylint-dev:main Oct 21, 2022
@cdce8p cdce8p deleted the contributors-relative-paths branch October 21, 2022 11:53
@cdce8p cdce8p added this to the 2.12.13 milestone Oct 21, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Maintenance Discussion or action around maintaining astroid or the dev workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants