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

chore: update .gitignore by cuining · Pull Request #3557 · coder/code-server · GitHub

/ code-server Public

chore: update .gitignore - #3557

Merged
jsjoeio merged 5 commits into
coder:mainfrom
cuining:main
Jun 8, 2021
ConversationCommits (5)ChecksFiles changed
Merged

chore: update .gitignore#3557
jsjoeio merged 5 commits into
coder:mainfrom
cuining:main

Conversation

cuining commented Jun 7, 2021

Copy link
Copy Markdown
Contributor

Checklist

  • updated CHANGELOG.md

out* will cause these below files to be ignored:

lib/vscode/src/vs/editor/contrib/documentSymbols/outlineModel.ts
lib/vscode/src/vs/editor/contrib/documentSymbols/test/
lib/vscode/src/vs/workbench/contrib/codeEditor/browser/outline/ lib/vscode/src/vs/workbench/contrib/notebook/browser/contrib/outline/
lib/vscode/src/vs/workbench/contrib/notebook/browser/view/output/
lib/vscode/src/vs/workbench/contrib/outline/
lib/vscode/src/vs/workbench/contrib/output/
lib/vscode/src/vs/workbench/services/outline/
lib/vscode/src/vs/workbench/services/output/

cuining requested a review from a team as a code owner June 7, 2021 06:34
jsjoeio self-assigned this Jun 7, 2021
jsjoeio added chore Related to maintenance or clean up new contributor For PRs by first-time contributor labels Jun 7, 2021

repo-ranger Bot commented Jun 7, 2021

Copy link
Copy Markdown
Contributor

Thanks for making your first contribution! 🙂

jsjoeio commented Jun 7, 2021

Copy link
Copy Markdown
Contributor

Wow, nice catch. Thanks so much for the PR @cuining!

RE: updated CHANGELOG.md

Totally optional but if you want to add this to the CHANGELOG.md then we'll credit you in the release notes for the next release!

cuining changed the title fix .gitignore chore: update .gitignore Jun 8, 2021
cuining requested a review from jsjoeio June 8, 2021 15:35

codecov Bot commented Jun 8, 2021

Copy link
Copy Markdown

Codecov Report

Merging #3557 (eadd194) into main (ecbef27) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3557   +/-   ##
=======================================
  Coverage   59.21%   59.21%           
=======================================
  Files          35       35           
  Lines        1709     1709           
  Branches      379      379           
=======================================
  Hits         1012     1012           
  Misses        559      559           
  Partials      138      138

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecbef27...eadd194. Read the comment docs.

jsjoeio merged commit 9052a13 into coder:main Jun 8, 2021
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

jsjoeio Awaiting requested review from jsjoeio
1 more reviewer
oxy oxy approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

jsjoeio

Labels

chore Related to maintenance or clean up new contributor For PRs by first-time contributor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL