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

Strip compilation date from doc outputs #4334

Merged
merged 5 commits into from
Apr 15, 2024
Merged

Strip compilation date from doc outputs #4334

merged 5 commits into from
Apr 15, 2024

Conversation

KrystalDelusion
Copy link
Member

@KrystalDelusion KrystalDelusion commented Apr 12, 2024

  • Fix yosys-abc usage changing for every compile.
  • Fix test*.log changing for every compile.
  • Fix test*.log changing for every verific release.
  • Resolve ambiguity on source/temp by renaming to source/generated (i.e. the files should not be included in the Yosys repo, but they are needed for publishing the docs).

@KrystalDelusion KrystalDelusion requested a review from mmicko April 12, 2024 23:35
Copy link
Member

@mmicko mmicko left a comment

Choose a reason for hiding this comment

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

:end-before: UC Berkeley needs to be removed
otherwise output is not generated :

docs/source/appendix/auxprogs.rst:39: WARNING: end-before pattern not found: UC Berkeley

Are logs actually used anywhere ?

I have added ignore file in tree so it should not affect builds.

e2cfcbc

@KrystalDelusion KrystalDelusion requested a review from mmicko April 14, 2024 21:51
@KrystalDelusion
Copy link
Member Author

The log files are not currently being used, but there is a todo in the page on extensions to use them instead of the inline code sections currently there.

.. todo:: update to use :file:`/code_examples/extensions/test*.log`

@mmicko mmicko merged commit 7bb2746 into main Apr 15, 2024
35 checks passed
@mmicko mmicko deleted the docs_tidy branch April 15, 2024 06:27
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.

2 participants