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

Change README default format to Markdown and add it to pyproject.toml automatically #294

Closed
wants to merge 4 commits into from
Closed

Conversation

cauebs
Copy link
Contributor

@cauebs cauebs commented Jul 9, 2018

  • Added tests for changed code.
  • Updated documentation for changed code.

Closes #280.

The problem right now is that I cannot change the pyproject.toml's structure, which is created here: https://github.com/cauebs/poetry/blob/a8b534811c1e11541673eb426d34ef9b824321d4/poetry/layouts/layout.py#L84-L89

I don't find the current solution very elegant. If I want to optionally add the readme to the default pyproject.toml, we'll end up with four different strings (with license and readme, just readme, just license or neither). I could edit the strings to insert a line in the correct position, but that's ugly as well. This TOML library is becoming very inconvenient. How should I deal with this?

@sdispater
Copy link
Member

Just so you know I am currently developing a TOML library which should ease the programmatic creation/modification of the pyproject.toml file.

I'll let you know when it's ready (hopefully it should be this week).

@cauebs
Copy link
Contributor Author

cauebs commented Jul 9, 2018

Awesome. Let me know if you want some help ;)

@sdispater
Copy link
Member

@cauebs I merged the PR with the new TOML library (TOML Kit) which should help with interacting with the pyproject.toml file.

@brycedrennan brycedrennan added the kind/feature Feature requests/implementations label Aug 15, 2019
@stale
Copy link

stale bot commented Nov 13, 2019

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Nov 13, 2019
@sdispater sdispater added stale and removed wontfix labels Nov 14, 2019
@stale stale bot removed the stale label Nov 14, 2019
@finswimmer
Copy link
Member

Close in favor of #1515

@finswimmer finswimmer closed this Nov 19, 2019
Copy link

github-actions bot commented Mar 1, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/feature Feature requests/implementations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default README format to Markdown in "new"?
4 participants