From 2a91c73fbc5a15631eb4fb3f4578ee0aa512f2dd Mon Sep 17 00:00:00 2001 From: markzegarelli Date: Fri, 1 Oct 2021 10:31:10 -0700 Subject: [PATCH 1/3] ready for default branch switch --- .github/workflows/repo-sync.yml | 6 +++--- CONTRIBUTING.md | 4 ++-- README.md | 10 +++++++--- devguide.md | 4 ++-- 4 files changed, 14 insertions(+), 10 deletions(-) diff --git a/.github/workflows/repo-sync.yml b/.github/workflows/repo-sync.yml index 5d28130c3b..2d420ddd84 100644 --- a/.github/workflows/repo-sync.yml +++ b/.github/workflows/repo-sync.yml @@ -17,7 +17,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.DOCS_BOT_PAT_REPO_SCOPE }} with: source_repo: ${{ secrets.SOURCE_REPO }} - source_branch: master + source_branch: ${{ secrets.SOURCE_BRANCH }} destination_branch: ${{ secrets.INTERMEDIATE_BRANCH }} github_token: ${{ secrets.DOCS_BOT_PAT_REPO_SCOPE }} @@ -27,7 +27,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.DOCS_BOT_PAT_REPO_SCOPE }} with: source_branch: ${{ secrets.INTERMEDIATE_BRANCH }} - destination_branch: master + destination_branch: ${{ secrets.DESTINATION_BRANCH }} pr_title: 'repo sync' pr_body: "This is an automated pull request to sync changes between the public and private repos.\n\n:robot: This pull request should be merged (not squashed) to preserve continuity across repos, so please let a bot do the merging!" pr_label: autoupdate,automated-reposync-pr @@ -39,7 +39,7 @@ jobs: with: github-token: ${{ secrets.GITHUB_TOKEN }} branch: repo-sync - base: master + base: ${{ secrets.DESTINATION_BRANCH }} author: bot-docsteam state: open diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f97fdfc279..4867c1bf5d 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -14,12 +14,12 @@ Not all pages have a 1-1 mapping with their location within the repository. This ## Want to go deeper? Fork the repository -You can fork this repository to your local machine to make larger changes. Examples of larger changes include: +You can fork this repository and clone it to your local machine to make larger changes. Examples of larger changes include: - editing more than one file at a time - adding or updating images - updating navigation items -In this scenario, you'll fork the repository locally, make your changes, and submit a pull request to have them reviewed and merged back into the site. +In this scenario, you'll fork the repository, clone it locally, make your changes, and submit a pull request to have them reviewed and merged back into the site. ## Site structure diff --git a/README.md b/README.md index cefb93e0d7..60f6a81107 100644 --- a/README.md +++ b/README.md @@ -16,7 +16,9 @@ In this article, find information about: ## Contributing -The Segment docs team accepts a few types of contribution, including: +The Segment docs team accepts contributions in the form of issues and pull requests. + +All accepted contributions, in the form of approved pull requests and resolved issues, are deployed twice weekly, on Tuesday and Thursdays. As a result, once your pull request is closed, or your issue resolved, you may not see these changes on the site immediately. ### Issues @@ -24,9 +26,11 @@ If you see an error or something incorrect on a specific page, click Request doc ### Pull Requests -If you would like to update the documentation yourself, create a Pull Request to bring it to the team’s attention. Click Edit this page to edit the page directly on GitHub. Alternatively, you can clone the repository and submit your Pull Request that way. For more information, see Contributing and the Developer Guide +If you would like to update the documentation yourself, create a Pull Request to bring it to the team’s attention. Click Edit this page to edit the page directly on GitHub. Alternatively, you can clone the repository and submit your Pull Request that way. For more information, see [Contributing](CONTRIBUTING.md) and the [Developer Guide](devguide.md) + + + -For more information about contributing, see [Contributing](CONTRIBUTING.md). ## List of READMEs diff --git a/devguide.md b/devguide.md index d80d64410a..c755e89596 100644 --- a/devguide.md +++ b/devguide.md @@ -16,7 +16,7 @@ If using OSX: ## All about the Catalog script -You run the Catalog update script by running `make catalog` from the docs repo. You, a person who is going to run the script, must first save a Segment token to an `.env` file locally, which is `gitignored` so we don’t check it in to gihub accidentally. +You run the Catalog update script by running `make catalog` from the docs repo. You, a person who is going to run the script, must first save a Segment token to an `.env` file locally, which is `gitignored` so we don’t check it in to GitHub accidentally. Note: Old ConfigAPI tokens are not compatible with Public API. You'll need a new one if you want to use Public API. @@ -33,7 +33,7 @@ It also does some slugification and destination-name normalization, since our ha ### 3.1. Layouts -`default.html` is the base container through which all the individual other layouts are built to have the right title, seo, etc. The template inheritance is described in the diagram below. +`default.html` is the base container through which all the individual other layouts are built to have the right title, SEO, etc. The template inheritance is described in the diagram below. The `destination.html`, `source.html`, and `integration.html` templates contain the logic that runs the layouts for individual catalog pages. Storage/warehouses use the generic Integration right now because they don't need anything special. Set the layout in the Jekyll `_config.yml` file. From 668725627fede0b2759da740e425d22d660a77ee Mon Sep 17 00:00:00 2001 From: markzegarelli Date: Fri, 1 Oct 2021 14:21:26 -0700 Subject: [PATCH 2/3] update devguide --- devguide.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/devguide.md b/devguide.md index c755e89596..3c8ef38548 100644 --- a/devguide.md +++ b/devguide.md @@ -4,7 +4,7 @@ The contents of this guide will help you get up and running with the Segment Doc ## Local development with `ruby` and `node`, without Config API -If using OSX: +If using MacOS: * Install command line tools, `xcode-select --install` * Install `Ruby` >= 2.3.0 https://www.ruby-lang.org/en/documentation/installation/ * Ensure you're running `RubyGems` >= 2.5.0 by running `gem update --system` followed by `gem --version` From e02b3de9f1ad475fc6c88abfb1880b10e5d1db26 Mon Sep 17 00:00:00 2001 From: markzegarelli Date: Fri, 1 Oct 2021 14:24:37 -0700 Subject: [PATCH 3/3] update devguide --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 60f6a81107..31ac6465d5 100644 --- a/README.md +++ b/README.md @@ -3,6 +3,8 @@ Here, you can contribute to the Segment Documentation site. +**Note**: On Friday, October 1st, the default branch of this repository was changed from `master` to `develop`. If you have a fork of `segment-docs`, please ensure the default branch of the fork is updated accordingly. + ## Overview This repository contains the documentation website code and source files for https://segment.com/docs.