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

Revert last GitBook commit #239

Merged
merged 1 commit into from
Sep 13, 2019

Conversation

dpordomingo
Copy link
Contributor

required by #237

This PR reverts automatic changes done at c9cdd13 by @vcoisne.

Per later investigation, I confirmed that each edition made from GitBook UI (no matter what is edited) forces GitBook to parse the whole docs, and format them according to their internal (and not documented) rules.
Some uncontrolled changes, that were reverted in this PR are:

  • broken anchors,
  • shortened titles,
  • docs/CONTRIBUTING.md being moved into docs/learn-more/contributing.md,
  • code syntax defined as text instead of bash or powershell,

But this PR also modifies any other intentional change that could be done by @vcoisne at c9cdd13.
I tried to review the diff to find his edition, but I could not find any major change, so I wonder if the causing edition was a typo fix, or any other small change that I could not detect.

As discussed with @carlosms by slack, it would be better if we rollback the whole automatic commit, and re-do the intentional change again using the regular PullRequest flow, instead of using the GitBook UI, which is adding unexpected noise when reviewing PRs.


  • I have updated the CHANGELOG file according to the conventions in keepachangelog.com
  • This PR contains changes that do not require a mention in the CHANGELOG file

This reverts commit c9cdd13.
And also keeps architecture, faq and troubleshooting in learn-more
as it was suggested by GitBook edition

Signed-off-by: David Pordomingo <[email protected]>
@dpordomingo dpordomingo added the bug Something isn't working label Sep 11, 2019
@dpordomingo dpordomingo requested review from vcoisne and a team September 11, 2019 07:13
@dpordomingo dpordomingo self-assigned this Sep 11, 2019
@dpordomingo dpordomingo force-pushed the revert-gitbook-disaster branch from 5aea34e to ff28f27 Compare September 11, 2019 09:08
@dpordomingo dpordomingo merged commit 6a8a697 into src-d:master Sep 13, 2019
@dpordomingo dpordomingo deleted the revert-gitbook-disaster branch September 13, 2019 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants