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 #189, Virtualenv and Pipenv .gitignore support #190

Merged
merged 1 commit into from
Sep 1, 2021
Merged

Fix #189, Virtualenv and Pipenv .gitignore support #190

merged 1 commit into from
Sep 1, 2021

Conversation

paulober
Copy link
Contributor

@paulober paulober commented Jul 28, 2021

Structured .gitignore

Describe the contribution

Testing performed

  1. made a commit with the new .gitignore

Contributor Info - All information REQUIRED for consideration of pull request
Paul Oberosler, Individual

@jphickey
Copy link
Contributor

I don't think this really needs to be in the project .gitignore file. There are lots of different IDEs and they all leave different workspace/backup/etc files around. While in the past some editor-specific patterns may have gotten introduced, in general each developer should just make their own ignore file (typically under $HOME/.config/git/ignore) that reflects the remnants left around by their editor/IDE of choice.

@paulober
Copy link
Contributor Author

Ok, then I'll close this PR.

@paulober paulober closed this Jul 28, 2021
@jphickey
Copy link
Contributor

Ok, then I'll close this PR.

That was quick ... may want to get some other opinions on the matter before closing ...

I'm not necessarily opposed to the venv/pycache stuff, since its likely that these files will get created no matter what IDE the user chooses.

@paulober paulober reopened this Jul 28, 2021
@paulober
Copy link
Contributor Author

Ok, I just thought that would take care of the whole .gitignore change thing. I can just throw out the IDE thing in a new commit and leave the venv thing in.

@paulober paulober changed the title Fix #189, PyCharm, virtualenv and Pipenv .gitignore support Fix #189, Virtualenv and Pipenv .gitignore support Jul 31, 2021
@astrogeco astrogeco added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) community and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Aug 21, 2021
.gitignore Outdated Show resolved Hide resolved
@astrogeco
Copy link
Contributor

CCB:2021-08-25 APPROVED

  • remove .desktop from gitignore

@astrogeco astrogeco added the CCB:Approved Indicates approval by CCB label Aug 30, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate September 1, 2021 04:09
@astrogeco astrogeco merged commit 00005b9 into nasa:integration-candidate Sep 1, 2021
@paulober paulober deleted the fix-189-jetbrainside-venv-pipenv-support-gitignore branch September 1, 2021 10:18
astrogeco added a commit to nasa/cFS that referenced this pull request Sep 1, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Sep 1, 2021
**Combines**

nasa/cFE#1885,              v6.8.0-rc1+dev980
nasa/osal#1138,             v5.1.0-rc1+dev598
nasa/cFS-GroundSystem#195,  v2.2.0-rc1+dev63

**Includes**

*cFE*

nasa/cFE#1870, Add SB API test cases
nasa/cFE#1869, Add ES API test cases
nasa/cFE#1872, Add TBL API test cases
nasa/cFE#1871, Add FS API test cases
nasa/cFE#1860, Add Time Clock Test
nasa/cFE#1862, EVS coverage test
nasa/cFE#1876, SB test improvements
nasa/cFE#1865, CFE_TBL_Modified: Test CRC, updated flag
nasa/cFE#1881, Improve EVS code coverage
nasa/cFE#1877, add call to CFE_ES_ExitChildTask
nasa/cFE#1902, Incorrect OSAL Format in Users Guide Reference
nasa/cFE#1884, Improve FS coverage
nasa/cFE, Improve MSG branch coverage
nasa/cFE#1891, Improve resource ID branch coverage
nasa/cFE#1894, Improve SBR branch coverage
nasa/cFE#1896, Fix #1895, Improve TIME branch coverage
nasa/cFE#1904, Improve TBL code coverage
nasa/cFE#1864, Support custom PSP directory
nasa/cFE#1913, Update time tests to use bitmask check macros
nasa/cFE#1923, remove extra word in comment

*osal*

nasa/osal#1136, add bitmask assert macros

*cFS-GroundSystem*

nasa/cFS-GroundSystem#190, Fix #189, Virtualenv and Pipenv .gitignore support
nasa/cFS-GroundSystem#194, Fix doc, comment, and message typos

Co-authored-by: Jacob Hageman           <[email protected]>
Co-authored-by: Joseph Hickey           <[email protected]>
Co-authored-by: Alex Campbell           <[email protected]>
Co-authored-by: Ariel Adams             <[email protected]>
Co-authored-by: Jose F Martinez Pedraza <[email protected]>
Co-authored-by: Avi                     <[email protected]>
Co-authored-by: Paul                    <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CCB:Approved Indicates approval by CCB community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Virutualenv and Pipenv support within .gitignore
3 participants