Skip to content

chore: make certain dotfiles searchable by ripgrep #1475

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

Merged
merged 1 commit into from
May 29, 2021
Merged

chore: make certain dotfiles searchable by ripgrep #1475

merged 1 commit into from
May 29, 2021

Conversation

JohnVillalovos
Copy link
Member

By explicitly NOT excluding the dotfiles we care about to the
.gitignore file we make those files searchable by tools like ripgrep.

By default dotfiles are ignored by ripgrep and other search tools (not
grep)

Copy link
Member

@nejch nejch left a comment

Choose a reason for hiding this comment

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

Thanks! I have a feeling we might forget to maintain this list, but it can't hurt 😀 Maybe this can be improved by reducing dotfile clutter also, a few more tools now support pyproject.toml and some stuff can go in .github/, I'll push a follow-up PR.

Just saw we're missing a few entries though.

By explicitly NOT excluding the dotfiles we care about to the
.gitignore file we make those files searchable by tools like ripgrep.

By default dotfiles are ignored by ripgrep and other search tools (not
grep)
@codecov-commenter
Copy link

codecov-commenter commented May 29, 2021

Codecov Report

Merging #1475 (e4ce078) into master (90ecf2f) will decrease coverage by 4.53%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1475      +/-   ##
==========================================
- Coverage   90.91%   86.37%   -4.54%     
==========================================
  Files          73       73              
  Lines        4051     4052       +1     
==========================================
- Hits         3683     3500     -183     
- Misses        368      552     +184     
Flag Coverage Δ
cli_func_v4 80.47% <ø> (+<0.01%) ⬆️
py_func_v4 ?
unit 82.00% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gitlab/v4/objects/files.py 61.76% <0.00%> (-30.89%) ⬇️
gitlab/v4/objects/milestones.py 71.42% <0.00%> (-28.58%) ⬇️
gitlab/v4/objects/tags.py 63.88% <0.00%> (-25.00%) ⬇️
gitlab/utils.py 65.51% <0.00%> (-24.14%) ⬇️
gitlab/v4/objects/repositories.py 57.62% <0.00%> (-23.73%) ⬇️
gitlab/v4/objects/sidekiq.py 80.95% <0.00%> (-19.05%) ⬇️
gitlab/v4/objects/commits.py 78.26% <0.00%> (-15.95%) ⬇️
gitlab/v4/objects/snippets.py 82.92% <0.00%> (-14.64%) ⬇️
gitlab/v4/objects/clusters.py 85.71% <0.00%> (-14.29%) ⬇️
gitlab/v4/objects/environments.py 88.23% <0.00%> (-11.77%) ⬇️
... and 12 more

@nejch nejch enabled auto-merge May 29, 2021 20:14
@nejch nejch merged commit 861d3d2 into python-gitlab:master May 29, 2021
@JohnVillalovos JohnVillalovos deleted the jlvillal/gitignore branch May 30, 2021 04:00
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