Skip to content

Latest commit

 

History

History
91 lines (61 loc) · 3.16 KB

PR_REQS.md

File metadata and controls

91 lines (61 loc) · 3.16 KB

✅ Pull Request Requirements

Creating your dev branch

Your dev branch should follow the format <build-platform>-<prefix>-<desc>

Where:

  • build-platform is one of: 'aws', 'azure', 'local', 'global'
  • prefix is one of:
    • 'fix' -- for improvements to an existing recipe
    • 'new' -- for a new recipe
    • 'docs' -- for doc changes/additions
    • 'test' -- for workflow/unit test modifications/additions
    • 'refactor' -- for structural changes to project/repo
  • desc is a meaningful identifier (such as recipe name) in kebab-case
git fetch upstream main
git checkout -b <build-platform>-<prefix>-<desc> -t upstream/main

Improving a recipe

Recipe improvements can be:

  • Semantic -- to improve the instruction clarity
  • Enhancements -- to improve the performance of the recipe
  • Extensions -- to extend the recipe capability

For extensions, consider if it is better suited as a new recipe instead, that lists the existing recipe as a prerequisite step. For example, see how Jenkins + CodePipeline references and builds on the Hello, Candle on AWS! tutorial.

Creating a recipe

NB: Your recipe name must be unique

  1. Copy the recipe template into the relevant build platform directory (i.e. aws, azure, local)
  2. Rename as your kebab-case recipe-name.md.
  3. Add your recipe and link to the relevant build-platform/index.md file
  4. Add your recipe and link to SUMMARY.md

To include assets create a new folder assets/source/<recipe-name>/

Images must be in .png or .jpg format and added to assets/source/<recipe-name>/

Reviewing your changes

Build Cookbook

make cookbook

Preview Cookbook at localhost:8000

NB: use the preview to check formatting and test hyperlinks

make serve
# CTRL+C to close server

Commiting your changes

git add .
git commit -m "<prefix> <desc>"
git push origin <build-platform>-<prefix>-<desc>

Open a Pull Request

From your forked your-username/candle-cookbook repo >> Pull requests >> New pull request:

base repository: nogibjj/candle-cookbook, base: main <-- head repository: your-username/candle-cookbook, comapre: your-branch

You will be asked to complete the Pull Request Checklist.

Your PR must pass the Deploy CI/CD status check.

⚠️ IMPORTANT: Once you have submitted a PR do not push any further changes. If you wish to make edits, delete the PR and submit a new one.

❌ NEVER FORCE PUSH: i.e. git push origin my-branch --force