-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/add auto documentation #30
base: develop
Are you sure you want to change the base?
Conversation
REMINDER!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an interesting add. Could be convenient. Can you provide steps for testing so I can test this before it's merged? Or do you have a demo you can provide?
- name: Build docs | ||
run: cargo doc | ||
- name: Add redirect | ||
run: echo '<meta http-equiv="refresh" content="0;url=credit_card_convenience/index.html">' > target/doc/index.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the significance of credit_card_convenience/index.html
?
I am requesting a pull into the master branch, but for good reason I think.
The only updates in this pr are twofold,
1st, An update to the readme.md with a little badge that says documentation that contains a hyperlink to the MOST LIKELY URL for the documentation (I had to guess at this since I don't have access to your account, but it should probably work)
2nd, a new github actions workflow has been added to the folder, its rather simple I think but I got it running on some other code, basically it will automatically run 'cargo doc' and then take the generated website and throw it into github pages.
NOTE: SETTINGS MIGHT NEED MODIFICATION on the repo's settings panel make sure that github pages has its source set to "Github Actions" This needs to be done PRIOR TO MERGING