Skip to content

DOC: start 1.0.4 #33970

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 2 commits into from
May 5, 2020
Merged

Conversation

simonjayhawkins
Copy link
Member

NOTE: opened against 1.0.x branch as to not commit to release at this stage.

I think could backport a few PRs moving the whatsnews and only after a release remove from 1.1 whatsnew on master. This gives us the chance to easily abandon the release at any time.

@simonjayhawkins simonjayhawkins added this to the 1.0.4 milestone May 4, 2020
@jreback
Copy link
Contributor

jreback commented May 4, 2020

lgtm. feel free to merge things as needed to this branch.

@simonjayhawkins simonjayhawkins merged commit 15ac3e2 into pandas-dev:1.0.x May 5, 2020
@simonjayhawkins simonjayhawkins deleted the DOC--start-1.0.4 branch May 5, 2020 11:26
@jorisvandenbossche
Copy link
Member

This needed to be merged in master first / as well (or did that already happen in another PR?)

@simonjayhawkins
Copy link
Member Author

This needed to be merged in master first / as well (or did that already happen in another PR?)

purposefully done it this way, see OP

NOTE: opened against 1.0.x branch as to not commit to release at this stage.

was planning to copy release notes on backports verbatim, then when ready to do a release, tidy up the whats new to be more applicable and then following a release, remove the applicable release notes from 1.1 whatsnew.

but could also work against master. wdyt?

@jorisvandenbossche
Copy link
Member

was planning to copy release notes on backports verbatim, then when ready to do a release, tidy up the whats new to be more applicable and then following a release, remove the applicable release notes from 1.1 whatsnew.

Yes, I think that is fine, but for backporting the PRs verbatim, you don't need this file (yet) in the 1.0.x branch, right?

But also in that workflow, the PR that fixes up the whatsnew files (moving the appropriate whatsnew entries from 1.1.0 to 1.0.4) can be done against master + backported?

@simonjayhawkins
Copy link
Member Author

Yes, I think that is fine, but for backporting the PRs verbatim, you don't need this file (yet) in the 1.0.x branch, right?

i meant copy the release note from 1.1 whatsnew to 1.0.4 whatsnew without changes in the backport PRs

so the 1.0.x branch doesn't have a 1.1. whatsnew and master doesn't have a 1.0.4 whatsnew (yet)

@jorisvandenbossche
Copy link
Member

Ah, yes, I forgot that the 1.0.x branch has no 1.1 file, so you can't just backport without changing the whatsnew. Yeah, then that workflow seems fine to me to move the whatsnew inside the backport PR

@simonjayhawkins
Copy link
Member Author

Yeah, then that workflow seems fine to me to move the whatsnew inside the backport PR

it does mean that each backport needs to be done manually, since auto backports fail. so there maybe a better workflow involving having a 1.1 whatsnew in 1.0.x temporarily.

the other downside is that any PRs in the pipeline that are suitable for backport can't add the release note to 1.0.4.

@jorisvandenbossche
Copy link
Member

jorisvandenbossche commented May 5, 2020

Was thinking the same and wondering if adding a "blank" 1.1.0 file (or only with the headers) would help, but it's quite possible that this would still give conflicts though in almost all cases in the whatsnew file.
But it could be worth to try out.

@jorisvandenbossche
Copy link
Member

(in general, for future releases, I think it will be easier to assume the bugfix release will happen, and when it's clear that it doesn't happen, it's easy to move the whatsnew entries from eg 1.0.4 to 1.1.0 and delete 1.0.4 file in one PR, instead of the other way around)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants