chevah / geck

Chevah Garden Of Eden Creation Kit
http://geck.chevah.com/
0 stars 1 forks source link

Remove Trac from our documentation. #82

Closed adiroiban closed 2 years ago

adiroiban commented 3 years ago

Scope

As we have migrated to GitHub, we no longer need to use Trac

Changes

Convert any Trac documentation into GitHub Issues

As a drive-by I have updated the site to latest python 3.8 and sphinx version

How to try and test the changes

reviewers: @danuker

Check that changes make sense

adiroiban commented 3 years ago

Can you please check that the content is ok

Dan, do you have a RTD account? I can add you to the geck project and you can try to see what is going on there.

needs-review

danuker commented 2 years ago

I created a RTD account; my login is danuker. You can add me if you want.

adiroiban commented 2 years ago

Done https://readthedocs.org/projects/chevah-geck/

danuker commented 2 years ago

changes-approved

Edit: oh, RobiNet did work, but my page did not update.

adiroiban commented 2 years ago

Thanks for the review.

I made some changes. Please review my responses.

This PR is about removing Trac references and not updating the whole dev and release process to GitHub tools.

The idea is that we will use GitHub only for some time, see what works best and then extract the best practices into geck.

needs-review

Thanks!

adiroiban commented 2 years ago

So the reason for the syntax change to [Fix #1234] is to auto-close them? Since the merge commit will include the PR title? If so, resolve this conversation.

Yes. The title of the PR is automatically used as the commit message.. and the commit message can automatically close an issue :)

adiroiban commented 2 years ago

i have reverted the brink.sh changes

needs-review

netlify[bot] commented 2 years ago

✔️ Deploy Preview for geck ready!

🔨 Explore the source changes: 9d2c0430f4a7684098f6337584519df38ea1ad89

🔍 Inspect the deploy log: https://app.netlify.com/sites/geck/deploys/62094b188945c600096638bb

😎 Browse the preview: https://deploy-preview-82--geck.netlify.app

adiroiban commented 2 years ago

I have merged this.

And migrated to netlify ... I don't know why :) ... I will revet to read the docs.