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.

Sign up for GitHub

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

Add requirements.txt file #10473

Closed
wants to merge 1 commit into from
+8 −0
Closed

Add requirements.txt file #10473

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
xmldiff
yamlpath
json2html
types-openpyxl
types-PyYAML
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like we'll need:

  • PyYAML
  • jinja2
  • pandas (although this is from the tool I'm writing, so I can add it there if this lands first)
  • tables
  • openpyxl
  • pygments

Copy link
Member Author

@Mab879 Mab879 Apr 19, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Looks like all of those packages are in the Fedora repos (and most are in RHEL). The current scope of this PR is only for packages not in Fedora. These would be good add to your PR in #10474.

PyYAML -> python3-pyyaml
jinja2 - > python3-jinja2
pandas -> python3-pandas
tables -> python3-tables That's not currently in our docs, can you cite which script needs this?
openpyxl -> python3-openpyxl
pygments -> python3-pygments, also not in our docs

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ok - good deal. Thanks.

Copy link
Member

Choose a reason for hiding this comment

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

Don't be shy and add them as well - if you use pip install on a system where part of the requirements are already satisfied, they are simply skipped.

openpyxl-stubs
cmakelint
-r docs/requirements.txt
Mab879 marked this conversation as resolved.
Show resolved Hide resolved