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

[REF] setup.cfg: Enable ChangeLog #6

Merged
merged 3 commits into from Nov 3, 2015

Conversation

moylop260
Copy link
Collaborator

No description provided.

@pedrobaeza
Copy link
Member

Where is ChangeLog file?

@moylop260
Copy link
Collaborator Author

Fixed

@moylop260
Copy link
Collaborator Author

FYI I can't verify If pypi is re-generating the ChangeLog
If we don't see a ChangeLog updated in pypi we will need add to .travis.yml file next line of code:
python setup.py sdist

@pedrobaeza
Copy link
Member

The ChangeLog is empty. For the regeneration question, you can see openupgradelib setup, that it's working fine putting changelog: https://github.com/OCA/openupgradelib/blob/master/setup.py#L38

@moylop260
Copy link
Collaborator Author

@pedrobaeza
If you download this repo and execute:
python setup.py sdist && cat ChangeLog you will see a non-empty file.
The idea is don't create it manually.

@moylop260
Copy link
Collaborator Author

The file auto-created is:

CHANGES
=======

* [ADD] ChangeLog: Empty file to support auto build
* [REF] setup.cfg: Enable ChangeLog

1.0.2
-----

* check that author is a string

1.0.1
-----

* [REF] Migrate md to rst

1.0.0
-----

* [IMP] .travis.yml: Enable auto deploy of pypi package from travis
* [ADD] pylint_odoo: Add main package
* [ADD] .gitignore: Add file gitignore for first commit

This information is auto-extracted from tags and commits messages.
I can avoid generate two times the same work.

@moylop260
Copy link
Collaborator Author

FYI I tested this deploy way in https://pypi.python.org/pypi/oca-pylint-plugin/1.11.2 of old project https://github.com/moylop260/pylint-oca
test

@pedrobaeza
Copy link
Member

OK, great

👍

@moylop260
Copy link
Collaborator Author

ping

@dreispt
Copy link
Sponsor Member

dreispt commented Nov 2, 2015

LGTM 👍

@moylop260
Copy link
Collaborator Author

@pedrobaeza
After merge this PR
Do you want have the honor of generate the first tag -s with this feature? (And other ones recently merged)

I proposal 1.1.0 like as @dreispt told us

pedrobaeza added a commit that referenced this pull request Nov 3, 2015
@pedrobaeza pedrobaeza merged commit 5227d8d into OCA:master Nov 3, 2015
@moylop260
Copy link
Collaborator Author

screenshot_2015-11-03-03-43-54

@pedrobaeza
Thank you

@luisg123v luisg123v deleted the master-oca-update-cfg branch August 6, 2020 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants