-
Notifications
You must be signed in to change notification settings - Fork 942
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
feature/readme migration v6 #1297
Conversation
this is great :) Since the migration guide is a very "temporary" item: in my eyes, it would go best on the Release page for v6.0.0, (similar to the v5.11.0 release page where I added a note about an exception). We could (and should) still link to it from the readme/docs. What do you think? If not, I think an .rst documentation file ultimately works fine :) |
I started writing in the README but it started to be so long I though it deserve its own space in the doc. We can keep it we never know when someone will upgrade to version 6, now or in a couple of years. In the readme I pan to add some section that has a link to the doc for the v6 migration |
I thought about it for some time, maybe we should keep it in the README which will force us to make it clear and short as well. you think it's a good idea ? |
That sounds like a good idea |
Signed-off-by: Alexandre Lavigne <lavigne958@gmail.com>
Signed-off-by: Alexandre Lavigne <lavigne958@gmail.com>
Signed-off-by: Alexandre Lavigne <lavigne958@gmail.com>
Signed-off-by: Alexandre Lavigne <lavigne958@gmail.com>
82bc30b
to
056eb28
Compare
rebased and updated the code so the migration guide is located in the README.md |
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.
This is good so far. I think we leave this open until we are ready to release 6.0.0 and then we can look at merging it.
I think the v6.0.0 migration
section should go between the Basic Usage
and More Examples
section. We should prioritise new users by showing the basics at the top of the README.
I disagree with this, we should merge this when it's ready so users can check it out, start planning and start updating what can be done. the best way would be: the users are ready to upgrade, then we make the new release and users can just merge their ready to go PR that includes code change and the bump of gspread version. because: a lot of users will be late 😆
agreed, make sense to show first the basic usage for the new users. 👍 I will update the doc with your suggestions |
Signed-off-by: Alexandre Lavigne <lavigne958@gmail.com>
agreed. I have made some changes, in effort to make the guide section of the README as small as possible. You can see further comments in #1279 (comment) I think that now we can merge this, and if we think of new breaking changes that should be added to the README, we can create a new PR linked to #1297 |
I added a whole new section in the docs that we can keep for some time.
It was way too big to be only in the README, it took too much space.
We can keep pushing on this branch until we are complete. what do you think @alifeee ?
Wtb at the end we need to add a link to the documentation in the README too but we can keep that for the last commit