Skip to content
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

fix: added .vscode and .nvmrc in .gitignore #4111

Merged
merged 2 commits into from
Feb 18, 2020
Merged

fix: added .vscode and .nvmrc in .gitignore #4111

merged 2 commits into from
Feb 18, 2020

Conversation

adrijshikhar
Copy link
Contributor

@adrijshikhar adrijshikhar commented Feb 17, 2020

Short description of what this resolves:

added .vscode in .gitignore

Checklist

  • I have read the Contribution & Best practices Guide.
  • My branch is up-to-date with the Upstream development branch.
  • The acceptance, integration, unit tests and linter pass locally with my changes

@codecov
Copy link

codecov bot commented Feb 17, 2020

Codecov Report

Merging #4111 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #4111   +/-   ##
============================================
  Coverage        21.88%   21.88%           
============================================
  Files              451      451           
  Lines             4729     4729           
============================================
  Hits              1035     1035           
  Misses            3694     3694

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 b990b6a...50b65d2. Read the comment docs.

@kushthedude
Copy link
Member

From next time please avoid micro-PRs. Bundle micro-PRs into a bigger one. We want to ensure that devs who review PRs use their time efficiently.

@@ -21,7 +21,7 @@ testem.log
/kubernetes/images/frontend/node_Modules/
.env
.idea

Copy link
Member

Choose a reason for hiding this comment

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

Don't remove this line.

@snitin315
Copy link
Member

Follow Semantic PR naming.

@iamareebjamal
Copy link
Member

Add both entries in a single PR

@adrijshikhar adrijshikhar changed the title added .vscode in .gitignore fix: added .vscode and .nvmrc in .gitignore Feb 18, 2020
@auto-label auto-label bot added the fix label Feb 18, 2020
iamareebjamal
iamareebjamal previously approved these changes Feb 18, 2020
@iamareebjamal iamareebjamal merged commit a9a4360 into fossasia:development Feb 18, 2020
@adrijshikhar adrijshikhar deleted the gitignore branch February 19, 2020 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants