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

documentation: fix a typo in Quickstart #50

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from

Conversation

jeras
Copy link

@jeras jeras commented Jun 26, 2022

I fixed a typo in a CLI and added a cosmetic update to a sentance.

@jeras
Copy link
Author

jeras commented Jun 26, 2022

I created this issue before I decided to fix the code and ask for a pull:
#49

@pawks
Copy link
Collaborator

pawks commented Jun 27, 2022

Thank you for the PR. A changelog update and version bump is needed to merge this. There seems to be residual mentions of arch-tests in commands.rst. Please replace those too.

@jeras
Copy link
Author

jeras commented Jun 27, 2022

Hi Pawan, I am reading the documentation further and will combine all the fixes into a single merge request with an updated CHANGELOG at the end. It will probably take me a week (I am trying to run tests on my CPU at the same time).

@jeras jeras force-pushed the master branch 3 times, most recently from eea0e76 to 5189ce1 Compare July 22, 2022 20:08
…hanges done to the entire documentation by search and manual replace
2. The database checks for macro sanity - i.e. certain macros exists and in the correct order.
3. Each time a new test is added to the ``suite`` directory, the database utility has to be run manually and the database.yaml
3. Each time a new test is added to the ``suite`` directory, the database utility has to be run manually and the ``database.yaml``
Copy link
Collaborator

Choose a reason for hiding this comment

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

This particular entry in the notes can be removed as the database is generated every time for each riscof run. It can probably be modified to:

3. The database file is generated by default every time for any relevant command unless a ``--database`` argument is supplied in the command. 
4. Each time a new test is added to the ``suite`` directory, the database utility has to be run  to ensure that the test is included in the ``database.yaml``. This can be done by running the `gendb` command or by skipping the ``--database`` argument in the CLI for relevant commands. 

Copy link
Author

Choose a reason for hiding this comment

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

I will have a look at it this weekend.

Otherwise I still have some documentation files to go through, before the changes would be ready for a merge.

I have split the changes into multiple commits, so they are easier to review.
Every find/replace (for capitalization) has its own commit, and I will try to split remaining changes into:

  • fixed width font,
  • spelling and grammar,
  • functionality.

I did not make changes to the Python source code, although there are many cases with missing capitalization.
Consequentially I also did not make changes to code blocks in the documentation.

@jamesbeyond jamesbeyond changed the base branch from master to dev May 21, 2024 17:01
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