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.

Sign up for GitHub

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

Jump to bottom

Change document structure #13

Merged
merged 9 commits into from
Oct 31, 2022
Merged

Change document structure #13

merged 9 commits into from
Oct 31, 2022

Conversation

bendnorman
Copy link
Member

No description provided.

zaneselvans
zaneselvans reviewed Oct 4, 2022
View reviewed changes
docs/bylaws.md Outdated Show resolved Hide resolved
zaneselvans
zaneselvans reviewed Oct 4, 2022
View reviewed changes
docs/policies.md Outdated
@@ -1,3 +1,54 @@
# Basic Expectations
Copy link
Member

Choose a reason for hiding this comment

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

"Basic" doesn't really provide any information. It seems like this is a reference to the number of hours per week / year we expect people to work. Can we make that explicit in the title and maybe move the Compensation section into Benefits (and make make it Compensation & Benefits?)

@zaneselvans
Copy link
Member

@bendnorman it looks like the docs build is failing because it's unable to solve the environment within RTD's memory constraints. It looks like it's installing libsnappy which shouldn't be required for this docs-only repo. Maybe there are other environment setup bits that could be simplified?

aesharpe
aesharpe reviewed Oct 25, 2022
View reviewed changes
Copy link
Member

@aesharpe aesharpe 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! One meta comment is that it would be nice to see some of the sub-headers in the menu on the left.

docs/catalyst_101.md Outdated Show resolved Hide resolved
docs/catalyst_101.md Outdated Show resolved Hide resolved
docs/catalyst_101.md Outdated Show resolved Hide resolved
docs/catalyst_101.md Outdated Show resolved Hide resolved
docs/articles_of_incorporation.md Show resolved Hide resolved
docs/catalyst_101.md Show resolved Hide resolved
docs/time_tracking.md Outdated Show resolved Hide resolved
docs/time_tracking.md Outdated Show resolved Hide resolved
docs/time_tracking.md Outdated Show resolved Hide resolved
docs/bylaws.md Show resolved Hide resolved
@bendnorman
Copy link
Member Author

bendnorman commented Oct 25, 2022 •
edited
Loading

Thank you for catching all of these typos and formatting issues!

@aesharpe
Copy link
Member

Thank you for catching all of these typos and formatting issues!

np! If you wouldn't mind marking the comments as resolved if/once you've addressed them that would be helpful!

@bendnorman bendnorman merged commit f02233d into main Oct 31, 2022
@bendnorman bendnorman deleted the catalyst-101 branch October 31, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aesharpe aesharpe aesharpe left review comments

@zaneselvans zaneselvans zaneselvans left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bendnorman @zaneselvans @aesharpe