-
Notifications
You must be signed in to change notification settings - Fork 201
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: Contribution Guide Updates #3766
Doc: Contribution Guide Updates #3766
Conversation
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
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 we going to create other versions of the deleted files, or is the goal to actually shrink and focus only on AVD-specific things?
Also, we should mention the minimum python version for development somewhere.
bef9f56
to
78c086f
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
ansible_collections/arista/avd/docs/contribution/development-tooling.md
Outdated
Show resolved
Hide resolved
…ooling.md Co-authored-by: Alexey Gorbunov <[email protected]>
Co-authored-by: Alexey Gorbunov <[email protected]>
Makefile
Outdated
@@ -44,24 +44,24 @@ pyavd-editable-install: ## Build and install PyAVD as editable | |||
######################################### | |||
|
|||
.PHONY: sanity | |||
sanity: sanity-info sanity-lint sanity-import ## Run ansible-test sanity validation. | |||
sanity: sanity-info sanity-lint sanity-import ## Run ansible-test sanity validation. Requires docker. |
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 still correct? Since you changed the default to venv.
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.
this target runs all 3 targets sanity-info sanity-lint sanity-import and sanity-info
requires docker. I will also update this and the document and makefile.
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.
nvm this comment, need more coffee this morning you are correct that we no longer require docker to run this!
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.
resolved with 69afeed
ansible_collections/arista/avd/docs/installation/collection-installation.md
Outdated
Show resolved
Hide resolved
…stallation.md Co-authored-by: Claus Holbech <[email protected]>
Quality Gate passedIssues Measures |
Change Summary
Makefile
:pre-commit-config.yml
.Remove unused/legacy content
Task List
Note: The Dev Container doesn't support running any test that requires docker, tracked in issue #4052