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

add tile author guide for new Kiln users #407

Merged
merged 19 commits into from
Jul 27, 2023
Merged

add tile author guide for new Kiln users #407

merged 19 commits into from
Jul 27, 2023

Conversation

crhntr
Copy link
Member

@crhntr crhntr commented Jul 1, 2023

This is a response to a request to help tile authors switch to Kiln from tile-generator or other tools and for existing kiln users to better understand and use the tool.

It has my opinions that are informed by PPE's kiln usage.

Some of the content in this file is already in the README. This is duplicate documentation but I think this is a good thing. The README is more in-depth; this is more beginner friendly.

@cf-gitbot
Copy link
Member

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

@crhntr crhntr changed the title add opinionated tile author guide Add opinionated tile author guide Jul 1, 2023
@crhntr crhntr changed the title Add opinionated tile author guide Add tile author guide for new Kiln users Jul 6, 2023
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
@crhntr crhntr marked this pull request as ready for review July 6, 2023 21:32
@crhntr crhntr requested a review from jaristiz July 7, 2023 18:16
@crhntr crhntr changed the title Add tile author guide for new Kiln users add tile author guide for new Kiln users Jul 10, 2023
@crhntr crhntr force-pushed the tile-author-guide branch 2 times, most recently from 0ddcae9 to df33da9 Compare July 12, 2023 16:37
TILE_AUTHOR_GUIDE.md Outdated Show resolved Hide resolved
Copy link
Contributor

@notrepo05 notrepo05 left a comment

Choose a reason for hiding this comment

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

Looks good

TILE_AUTHOR_GUIDE.md Show resolved Hide resolved
@notrepo05 notrepo05 dismissed jaristiz’s stale review July 27, 2023 17:29

Will address after merging

crhntr added 12 commits July 27, 2023 13:50
This guide is for new tile authors to get started or modernize with kiln built tiles.

squashed commits:
- add docs for testing and go import
- add heading for kiln test
- make authoring template parts more useful
- better document the bootstrapping section
- improve documentation for caching compiled releases
- add Stemcell Lock info to commit message
- add shell script command versions
- add curl version for completness
- clairify ominous comment about buildpacks
- document kiln update-release's ability to cause kiln validate to fail
- fix typo
- add documentation on what each kiln release source can do
    this is based on whether the release source implements component.ReleaseUploader
- add local tarballs section
- document kiln glaze
- replace some todos with first draft documentation
- use perm-links for example links
also fix typo in script
Thanks @spikymonkey for pointing out this was un-documented.
@crhntr crhntr merged commit d801d23 into main Jul 27, 2023
2 checks passed
@crhntr crhntr deleted the tile-author-guide branch July 27, 2023 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants