-
Notifications
You must be signed in to change notification settings - Fork 99
Update changelog #1611
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
Merged
Merged
Update changelog #1611
Changes from all commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
a4362f1
start updating changelog
aaronayres35 60b9dd4
group some changes under single bullet point
aaronayres35 14b5bd5
put pr numbers in ()
aaronayres35 f1d947c
add recent PRs
aaronayres35 eee3f21
pull news fragment system from apptools for use in future changelog u…
aaronayres35 cd410be
a couple more PRs and some highlights of the release
aaronayres35 0b873a4
more highlights of the release
aaronayres35 0521127
typos
aaronayres35 998dda3
editors can also be imported directly from traitsui.api
aaronayres35 902bf79
add recent PRs
aaronayres35 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
|
|
||
| **Checklist** | ||
| - [ ] Add a news fragment if this PR is news-worthy for end users. (see docs/releases/README.rst) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| The `upcoming` directory contains news fragments that will be added to the | ||
| changelog for the NEXT release. | ||
|
|
||
| Changes that are not of interest to the end-user can skip adding news fragment. | ||
|
|
||
| Add a news fragment | ||
| ------------------- | ||
| Create a new file with a name like ``<pull-request>.<type>.rst``, where | ||
| ``<pull-request>`` is a pull request number, and ``<type>`` is one of: | ||
|
|
||
| - ``feature``: New feature | ||
| - ``bugfix``: Bug fixes | ||
| - ``deprecation``: Deprecations of public API | ||
| - ``removal``: Removal of public API | ||
| - ``doc``: Documentation changes | ||
| - ``test``: Changes to test suite ('end users' are distribution packagers) | ||
| - ``build``: Build system changes that affect how the distribution is installed | ||
|
|
||
| Then write a short sentence in the file that describes the changes for the | ||
| end users, e.g. in ``123.removal.rst``:: | ||
|
|
||
| Remove package xyz. | ||
|
|
||
| Alternatively, use the following command, run from the project root directory | ||
| and answer the questions:: | ||
|
|
||
| python etstool.py changelog create | ||
|
|
||
| (This command requires ``click`` in the environment.) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which should we be recommending here? I listed both currently, but I imagine we just want people to import from
traitsui.api?This has been discussed before
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont remember that discussion - or the conclusion of that discussion.
I'm fine with listing both. I completely forgot about the fact that the editor factories are also importable via
traitsui.api. I think we can leave it up to the users to choose. I don't think we'll removetraitsui.editors.api.