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

Jump to bottom

restructure directories - move clab and act topo files #16

Merged
merged 5 commits into from
Jul 30, 2024
Merged

restructure directories - move clab and act topo files #16

merged 5 commits into from
Jul 30, 2024

Conversation

MitchV85
Copy link
Collaborator

@MitchV85 MitchV85 commented Jul 30, 2024 •
edited
Loading

Change Summary

  • Restructure directories in preparation for combining of Community labs and Tech Library labs
  • Move all clab related files and directories over from Tech Library repo
  • Update .gitignore to no longer exclude avd generated output configs and documentation
  • Create .gitattributes file to ensure no newline character issues occur with folks using Windows

Related Issue(s)

N/A

Repository Checklist

  • My code has been rebased from the upstream main branch

@MitchV85 MitchV85 requested a review from ankudinov July 30, 2024 00:48
@MitchV85 MitchV85 marked this pull request as ready for review July 30, 2024 01:57
ankudinov
ankudinov approved these changes Jul 30, 2024
View reviewed changes
Copy link
Contributor

@ankudinov ankudinov left a comment

Choose a reason for hiding this comment

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

LGTM

@MitchV85 MitchV85 merged commit f8dbf9c into aristanetworks:main Jul 30, 2024
1 check passed
@MitchV85 MitchV85 deleted the act-topo-updates branch July 30, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ankudinov ankudinov ankudinov approved these changes

Assignees
No one assigned
Labels
datacenter
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MitchV85 @ankudinov