FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore: make certain dotfiles searchable by ripgrep by JohnVillalovos · Pull Request #1475 · python-gitlab/python-gitlab · GitHub

Repository navigation

chore: make certain dotfiles searchable by ripgrep - #1475

Merged
nejch merged 1 commit into
python-gitlab:masterfrom
JohnVillalovos:jlvillal/gitignore
May 29, 2021
Merged

nejch merged 1 commit into
python-gitlab:masterfrom
JohnVillalovos:jlvillal/gitignore

Conversation

Copy link
Copy Markdown
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)

nejch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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.

Comment thread .gitignore
Comment thread .gitignore Outdated
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 commented May 29, 2021 •
edited
Loading

Copy link
Copy Markdown

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 enabled auto-merge May 29, 2021 20:14
nejch merged commit 861d3d2 into python-gitlab:master May 29, 2021
JohnVillalovos deleted the jlvillal/gitignore branch May 30, 2021 04:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL