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

Directory checks & print statement updates #34

Merged
merged 12 commits into from
Sep 4, 2023
Merged

Conversation

klmcadams
Copy link
Contributor

  • Updated README.rst to provide more details about how to set up your repository with reuse and how to run the hook on directories other than src
  • Removed linkcheck_ignore from conf.py since the repo is public
  • Added default_dir variable to easily change the default directory the loc argument is given
  • Added a check if the default_dir exists, and corresponding instructions if it doesn't exist
  • Added tests to check the above changes

@github-actions github-actions bot added documentation Improvements or additions to documentation maintenance Package and maintenance related labels Sep 1, 2023
@RobPasMue
Copy link
Member

I edited the README.rst file to render properly

image

And I also took advantage of the PR and added pre-commit.ci to it =)

Thanks for the PR @klmcadams!

@RobPasMue
Copy link
Member

Let me work on the checks... I might do some changes

Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM! Merging!

@RobPasMue RobPasMue enabled auto-merge (squash) September 4, 2023 08:07
@RobPasMue RobPasMue merged commit 79b59b3 into main Sep 4, 2023
25 checks passed
@RobPasMue RobPasMue deleted the maint/expand-output branch September 4, 2023 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants