Skip to content

Conversation

@AAYUSH2091
Copy link
Collaborator

  • Add matrix strategy to run tests for both collections
  • Support push/PR triggers on main, devel, and master branches
  • Parameterize working directories for collection-specific testing

@AAYUSH2091 AAYUSH2091 force-pushed the Add_mutli_muticollection_ci_support branch 2 times, most recently from eb95a58 to b4bc424 Compare December 8, 2025 10:16
@AAYUSH2091 AAYUSH2091 force-pushed the Add_mutli_muticollection_ci_support branch from 6e66601 to 8216fc6 Compare December 8, 2025 12:20
@AAYUSH2091 AAYUSH2091 marked this pull request as ready for review December 8, 2025 12:41
Comment on lines +1 to +9
#
# (c) 2016 Red Hat Inc.
#
# This file is part of Ansible
#
# Ansible is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don;t need action plugin here , we can simple add redirection for it

@@ -1,5 +1,354 @@
from ansible_collections.juniper.device.plugins.cliconf.junos import Cliconf
#
Copy link
Collaborator

Choose a reason for hiding this comment

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

we simple need to have header and doc nothing else

@rohitthakur2590
Copy link
Collaborator

@AAYUSH2091 thanks for these changes here is another PR #796
I have taken all your workflow changes and have improved with the comments I have added here.

If you want you can align your PR with #796 or we can close this one and merge #796

@AAYUSH2091
Copy link
Collaborator Author

Closing this PR in favor of #796

@AAYUSH2091 AAYUSH2091 closed this Dec 17, 2025
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