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

Contributing to the Wiki #600

Closed
mottosso opened this issue Apr 18, 2019 · 7 comments
Closed

Contributing to the Wiki #600

mottosso opened this issue Apr 18, 2019 · 7 comments

Comments

@mottosso
Copy link
Contributor

Hi all,

I've added a page to the Wiki on releasing packages, and have a few ideas on how to improve the experience.

The Problem

  1. Edits to the wiki go unnoticed, no notifications. :(
  2. Edits cannot be part of a PR with related code, and thus not commented on or iterated on collaboratively
  3. The Wiki is versioned, but separately from the parent repo git commits
  4. The Wiki and current version of Rez are disconnected; meaning that if you run an older version of Rez, you've got no documentation. The Wiki is only able to display its contents at a single point in time.

A solution(s)

  1. Store the Wiki in the parent repo, and automagically deploy it on push; e.g. rez\wiki. That solves (1), (2) and (3), in that we can now make PRs with both code and documentation changes. Win! But it doesn't solve (4), which is a bummer..
  2. Host the documentation in the parent rez repo, but publish it elsewhere, like readthedocs. Then each version could get tagged along with a particular version of the Rez source, and updates could be made retroactively to an older version, whilst still displaying multiple versions of the docs to the end-user. Win!

An example of docs with multiple versions.

See top of page.

@bpabel
Copy link
Contributor

bpabel commented Apr 18, 2019

+1 for readthedocs. Especially if docs can be migrated to something like sphinx that can take advantage of autodoc features.

@uncojohnco
Copy link

uncojohnco commented Apr 18, 2019

+1 readthedocs

@nerdvegas
Copy link
Contributor

nerdvegas commented Apr 19, 2019 via email

@mottosso
Copy link
Contributor Author

Please don't update the wiki directly ... There are a few aspects that I
don't think quite convey the role of rez-release correctly though ... not quite correct

No worries, was expecting my blunt initiative and strong assertions to prompt clear direction and corrections so this is great. :)

That wiki/ folder is great and already accounts for problems 1-3, and I'm not too fussed about multiple version support at the moment myself.

@nerdvegas
Copy link
Contributor

Closing for now, readthedocs would be nice but will open separate enhancement issue for this at a later date

@mottosso
Copy link
Contributor Author

I had a look at contributing via the wiki/ directory, but it doesn't seem up to date with the current version.

I suspect this hasn't been pushed to the Wiki for some time. I would update it, but since edits have been disabled, fetching the original markdown is unavailable too.

For my future self, what I'm looking to update is the section about package_filter, which currently only contains a YAML reference (despite surrounding topics being Python) (and which I'm not sure how use)

  1. How to use it with rezconfig.py
  2. How to use it with an environment variable.

rezconfig.py

package_filter = {'excludes': 'glob(*-beta*)'}

environment

set REZ_PACKAGE_FILTER_JSON="{'excludes': 'glob(*-beta*)'}"

@nerdvegas
Copy link
Contributor

The "Configuring Rez" page is a special case - its content is derived directly from rezconfig.py.

Note this comment in rezconfig.py:

Note: The comments in this file are extracted and turned into Wiki content. Pay
attention to the comment formatting and follow the existing style closely.

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

No branches or pull requests

4 participants