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

Miscellaneous docs fixes #378

Merged
merged 4 commits into from
Sep 21, 2023
Merged

Miscellaneous docs fixes #378

merged 4 commits into from
Sep 21, 2023

Conversation

btjanaka
Copy link
Member

Description

Misc fixes to docs.

TODO

  • Install all in addition to dev in CONTRIBUTING so that tests pass
  • Fix PR format for a line in HISTORY.md

Questions

Status

  • I have read the guidelines in
    CONTRIBUTING.md
  • I have formatted my code using yapf
  • I have tested my code by running pytest
  • I have linted my code with pylint
  • [N/A] I have added a one-line description of my change to the changelog in
    HISTORY.md
  • This PR is ready to go

@btjanaka btjanaka merged commit e445256 into master Sep 21, 2023
17 checks passed
@btjanaka btjanaka deleted the docs-fixes branch September 21, 2023 22:44
btjanaka added a commit that referenced this pull request Sep 22, 2023
## Description

<!-- Provide a brief description of the PR's purpose here. -->

Misc fixes to docs.

## TODO

<!-- Notable points that this PR has either accomplished or will
accomplish. -->

- [x] Install `all` in addition to `dev` in CONTRIBUTING so that tests
pass
- [x] Fix PR format for a line in HISTORY.md

## Questions

<!-- Any concerns or points of confusion? -->

## Status

- [x] I have read the guidelines in

[CONTRIBUTING.md](https://github.com/icaros-usc/pyribs/blob/master/CONTRIBUTING.md)
- [x] I have formatted my code using `yapf`
- [x] I have tested my code by running `pytest`
- [x] I have linted my code with `pylint`
- [N/A] I have added a one-line description of my change to the
changelog in
      `HISTORY.md`
- [x] This PR is ready to go
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.

1 participant