-
Notifications
You must be signed in to change notification settings - Fork 31
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
update docs and dependencies #223
Conversation
| Device Type | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :x: | | ||
| Mgmt Interface | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :x: | | ||
| Mgmt IP Address | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :white_check_mark: | :x: | | ||
| Hostname | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ❌ | |
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.
Can we use the actual unicode/emoji characters ✅ ❌ 🧪 here? For example:
| Hostname | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ❌ | | |
| Hostname | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ❌ | |
(see also #218).
@@ -0,0 +1 @@ | |||
Remove secrets-provider app dependency update doc dependencies |
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.
Remove secrets-provider app dependency update doc dependencies | |
Removed secrets-provider app dependency. | |
Updated documentation dependencies. |
"""Verify that pyproject.toml dev dependencies have the same versions as in the docs requirements.txt.""" | ||
parent_path = os.path.dirname(os.path.dirname(os.path.dirname(os.path.realpath(__file__)))) | ||
parent_path = os.path.dirname( | ||
os.path.dirname(os.path.dirname(os.path.realpath(__file__))) | ||
) | ||
poetry_path = os.path.join(parent_path, "pyproject.toml") | ||
poetry_details = toml.load(poetry_path)["tool"]["poetry"]["group"]["dev"]["dependencies"] | ||
with open(f"{parent_path}/docs/requirements.txt", "r", encoding="utf-8") as file: | ||
requirements = [line for line in file.read().splitlines() if (len(line) > 0 and not line.startswith("#"))] | ||
poetry_details = toml.load(poetry_path)["tool"]["poetry"]["group"]["dev"][ | ||
"dependencies" | ||
] | ||
with open( | ||
f"{parent_path}/docs/requirements.txt", "r", encoding="utf-8" | ||
) as file: | ||
requirements = [ | ||
line | ||
for line in file.read().splitlines() | ||
if (len(line) > 0 and not line.startswith("#")) | ||
] |
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.
Is this just a formatting change? Did the poetry.lock updates include updates to ruff, or why was this necessary?
@@ -66,11 +66,15 @@ def is_truthy(arg): | |||
|
|||
|
|||
def _is_compose_included(context, name): |
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.
Are all of the below changes just due to a different version of ruff?
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.
The scope of this PR is too large and includes things that overlap with drift manager. I will close it.
Remove the secrets-providers app as a dependency
Update readme to fix icons
Add/Update docs dependencies to match core
Increase Nautobot version for local development and CI to 2.3.0