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

DOC: Changes documentation to mkdocs #2419

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

gethvi
Copy link
Contributor

@gethvi gethvi commented Oct 31, 2023

@gethvi gethvi self-assigned this Oct 31, 2023
@gethvi gethvi marked this pull request as draft October 31, 2023 16:08
@gethvi gethvi changed the title Changes documentation to mkdocs DOC: Changes documentation to mkdocs Nov 1, 2023
@gethvi gethvi force-pushed the mkdocs branch 14 times, most recently from d5f51da to cb0df01 Compare November 2, 2023 12:25
@gethvi gethvi marked this pull request as ready for review November 2, 2023 13:08
setup.py Show resolved Hide resolved
@gethvi gethvi force-pushed the mkdocs branch 4 times, most recently from 4bfc672 to f3cdab6 Compare November 3, 2023 16:27
Copy link
Contributor

@kamil-certat kamil-certat left a comment

Choose a reason for hiding this comment

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

I haven't checked all changes in the docs (this will come with the usage, but I feel you also addressed some my concerns about the docs structure :D). I love the change, and want to see it online <3

However, could you please bring back to the README two things:

  • IntelMQ logo (because it looks sad on GitHub without it)
  • European funding info (because I'm pretty sure there are still active legal requirements to say it clearly).

I'd also add a link to docs in the README - even if it would be redundant on the docs website, it's important in GitHub

.github/workflows/build-docs.yml Show resolved Hide resolved
Makefile Show resolved Hide resolved
mkdocs.yml Outdated Show resolved Hide resolved
@aaronkaplan aaronkaplan merged commit d693561 into certtools:develop Nov 6, 2023
20 checks passed
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.

4 participants