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 DEVELOPMENT.md #901

Merged
merged 2 commits into from
Aug 17, 2018
Merged

Conversation

bobcatfish
Copy link
Contributor

This DEVELOPMENT.md will hopefully help folks ramp up and get started
contributing to skaffold by describing the development process.

After discussions with @r2d4 and @balopat we want to try out keeping the
contributor/developer docs in markdown, while the user facing docs will
continue to be in asciidoc. One of the reasons for this is that github
will automatically link to the CONTRIBUTING.md and code of conduct, and
so if we wrote this in ascii doc, we would have to make the docs (and
any references, links, etc.) that they contain work for both markdown
view in github and ascii doctor view in the generated docs.

Finally made this PR!!!! Sorry for the delay @r2d4 @balopat 😅

This DEVELOPMENT.md will hopefully help folks ramp up and get started
contributing to skaffold by describing the development process.

After discussions with @r2d4 and @balopat we want to try out keeping the
contributor/developer docs in markdown, while the user facing docs will
continue to be in asciidoc. One of the reasons for this is that github
will automatically link to the CONTRIBUTING.md and code of conduct, and
so if we wrote this in ascii doc, we would _have_ to make the docs (and
any references, links, etc.) that they contain work for both markdown
view in github and ascii doctor view in the generated docs.
make docs
```

And then open the generated docs/generated folder for `index.html` and `index.pdf`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

One part of this process I'm still not clear on: now that I've made doc changes, do I need to do anything special to publish those?

Copy link
Contributor

Choose a reason for hiding this comment

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

no, docs automatically get published after changes are merged to master in the latest builds bucket on https://storage.googleapis.com/skaffold/builds/latest/docs/index.html. When we do a release it gets released into the given versions bucket: https://storage.googleapis.com/skaffold/releases/latest/docs/index.html

Currently we don't host these on anywhere else than GCS.

@codecov-io
Copy link

codecov-io commented Aug 16, 2018

Codecov Report

Merging #901 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #901   +/-   ##
=======================================
  Coverage   38.27%   38.27%           
=======================================
  Files          56       56           
  Lines        2576     2576           
=======================================
  Hits          986      986           
  Misses       1476     1476           
  Partials      114      114

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd9fb3c...dc19b63. Read the comment docs.

Copy link
Contributor

@balopat balopat left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you very much for improving these!

DEVELOPMENT.md Outdated
@@ -120,6 +120,9 @@ make docs

And then open the generated docs/generated folder for `index.html` and `index.pdf`.

Once PRs with doc changes are merged, the docs will get automatically published to
[the public docs](https://storage.googleapis.com/skaffold/builds/latest/docs/index.html).
Copy link
Contributor

Choose a reason for hiding this comment

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

do we want to point out that there are separate builds and releases?

@balopat balopat merged commit 90cdfd4 into GoogleContainerTools:master Aug 17, 2018
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.

3 participants