Skip to content

Renamed cookiecutter.package to API#1442

Merged
ssbarnea merged 2 commits intocookiecutter:masterfrom
usneil:cookiecutterpkg-rename-doc2-0
Aug 15, 2020
Merged

Renamed cookiecutter.package to API#1442
ssbarnea merged 2 commits intocookiecutter:masterfrom
usneil:cookiecutterpkg-rename-doc2-0

Conversation

@usneil
Copy link
Copy Markdown
Contributor

@usneil usneil commented Aug 15, 2020

Fixes #1410

Copy link
Copy Markdown
Member

@ssbarnea ssbarnea left a comment

Choose a reason for hiding this comment

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

I am pretty sure this should have not passed CI as its it required for the title line to match the text length.

Clearly we missed to lint this.

matching line with its corresponding text length
@usneil
Copy link
Copy Markdown
Contributor Author

usneil commented Aug 15, 2020

right, maybe the CI test missed it. I have updated the line length, hope it suffices. Thanks for notifying @ssbarnea

@ssbarnea
Copy link
Copy Markdown
Member

It was never checked, I proposed a PR to fix few minutes ago. There were few issues I uncovered.

@usneil
Copy link
Copy Markdown
Contributor Author

usneil commented Aug 15, 2020

got it. so would this PR be sufficient to close issue #1410?

@ssbarnea ssbarnea added the documentation This issue/PR relates to or includes documentation. label Aug 15, 2020
@ssbarnea ssbarnea merged commit b155476 into cookiecutter:master Aug 15, 2020
@simobasso simobasso mentioned this pull request May 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This issue/PR relates to or includes documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Documentation 2.0: cookiecutter.package menu should be renamed to API

2 participants