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

Cut over to the new spec authoring platform #3094

Merged
merged 10 commits into from
Apr 6, 2021
Merged

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Apr 5, 2021

Should check off enough of #3055 to unfreeze. Incorporates much of #3025, though we may still want to look at #3025 for supplemental documentation.

In collaboration with matrix-org/matrix.org#970


Reviewer: The commits are largely isolated and have a bit of a story progression to them. This isn't a perfect cleanup, but it should be enough to continue spec writing.


The CI is not transitioned fully here, but the now-wasted bytes from the bundle are removed.

Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great on the whole. Just a bunch of little fixes and suggestions below.

.buildkite/pipeline.yaml Show resolved Hide resolved
CONTRIBUTING.rst Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
meta/documentation_style.rst Outdated Show resolved Hide resolved
.buildkite/pipeline.yaml Show resolved Hide resolved
.circleci/config.yml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@@ -337,7 +337,7 @@ func (s *server) serveSpec(w http.ResponseWriter, req *http.Request) {
log.Printf("Serving pr %s (%s)\n", branchName, sha)
} else if strings.ToLower(branchName) == "head" ||
branchName == "master" ||
strings.HasPrefix(branchName, "drafts/") {
strings.HasPrefix(branchName, "attic/drafts/") {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ctrl+f for the win

@turt2live turt2live merged commit 7a0f6aa into master Apr 6, 2021
@turt2live turt2live deleted the travis/cutover-1 branch April 6, 2021 15:51
@anoadragon453 anoadragon453 mentioned this pull request Apr 6, 2021
5 tasks
richvdh pushed a commit that referenced this pull request Aug 23, 2021
richvdh pushed a commit that referenced this pull request Aug 23, 2021
richvdh pushed a commit that referenced this pull request Aug 23, 2021
Cut over to the new spec authoring platform
richvdh pushed a commit that referenced this pull request Aug 23, 2021
richvdh pushed a commit that referenced this pull request Aug 27, 2021
richvdh pushed a commit that referenced this pull request Aug 27, 2021
richvdh pushed a commit that referenced this pull request Aug 27, 2021
Cut over to the new spec authoring platform
richvdh pushed a commit that referenced this pull request Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants