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

[13.0] cleanup gitignore, lib dir should not be ignored (used for javascript 3rd party libraries) #824

Closed
wants to merge 1 commit into from
Closed

Conversation

sherpya
Copy link
Member

@sherpya sherpya commented Apr 28, 2020

Most entries are only useful if developing python extensions
or using buildout
I've added also .vscode to ignore

@sherpya sherpya changed the title [13.0] cleanup gitignore, lib dir should not be ignored [13.0] cleanup gitignore, lib dir should not be ignored (used for javascript 3rd party libraries) Apr 28, 2020
Most entries are only useful if developing python extensions
or using buildout
I've added also .vscode to ignore
@alexey-pelykh
Copy link
Contributor

@simahawk
Copy link

Thanks for your proposal.

This is based on https://github.com/github/gitignore/blob/master/Python.gitignore and IMO there's no reason to drop those lines, but if we go for this change it must be done on all repos.

Anyway, editors should be ignored globally by your own config https://help.github.com/en/github/using-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer

If you want to propose common change, pls open a PR following @alexey-pelykh link.

@simahawk simahawk closed this Apr 30, 2020
@sherpya
Copy link
Member Author

sherpya commented Apr 30, 2020

I'd suggest making PR to the https://github.com/OCA/maintainer-quality-tools/blob/master/sample_files/.gitignore

this one is ok, at the end has

# OCA rules
!static/lib/

@sherpya
Copy link
Member Author

sherpya commented Apr 30, 2020

Thanks for your proposal.

This is based on https://github.com/github/gitignore/blob/master/Python.gitignore and IMO there's no reason to drop those lines, but if we go for this change it must be done on all repos.

Anyway, editors should be ignored globally by your own config https://help.github.com/en/github/using-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer

If you want to propose common change, pls open a PR following @alexey-pelykh link.

ok adding

# OCA rules
!static/lib/

instead?
alternatively should I need to temp disable git ignore to add lib in a module?

@simahawk
Copy link

I've proposed this OCA/maintainer-quality-tools#641
Normally we should not have any ext lib on python side included in the module so we tend to support it only for static folder.
You can review it and support it or propose another approach w/ proper motivation ;)

@sherpya
Copy link
Member Author

sherpya commented Apr 30, 2020

I've proposed this OCA/maintainer-quality-tools#641
Normally we should not have any ext lib on python side included in the module so we tend to support it only for static folder.
You can review it and support it or propose another approach w/ proper motivation ;)

I've recreated pr adding only:

# OCA rules
!static/lib/

alternatively please update the hr .gitgnore from https://github.com/OCA/maintainer-quality-tools/blob/master/sample_files/.gitignore as per @alexey-pelykh suggestion.

I really don't need the cleanup but only to add files in static/lib/

@sherpya
Copy link
Member Author

sherpya commented Apr 30, 2020

Anyway, editors should be ignored globally by your own config https://help.github.com/en/github/using-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer

I added only vscode, and the comment to eclipse (entries were altready in), pycharm and others were already in.

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.

None yet

3 participants