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

fix(pre_commit): DO NOT MERGE - Bumped pre_commit_terraform and pre_commit versions #53

Closed
wants to merge 2 commits into from
Closed

fix(pre_commit): DO NOT MERGE - Bumped pre_commit_terraform and pre_commit versions #53

wants to merge 2 commits into from

Conversation

horiagunica
Copy link
Contributor

@horiagunica horiagunica commented Dec 17, 2024 •
edited
Loading

Description

Issues with pre-commit fmt - bump versions.

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@horiagunica horiagunica requested a review from a team as a code owner December 17, 2024 08:40
@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/idempotence

Required input 'paths' not provided

@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/idempotence paths="examples/multi_nic_common examples/panorama_standalone examples/standalone_vmseries_with_metadata_bootstrap examples/vmseries_ha examples/vpc_peering_common examples/vpc_peering_common_with_autoscale examples/vpc_peering_common_with_network_tags examples/vpc_peering_dedicated examples/vpc_peering_dedicated_with_autoscale" tf_version="1.7 1.8"

Testing job ID: 12369422257
Job result: FAILURE

@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/sca

Testing job ID: 12369639830
Job result: FAILURE

@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/sca

Testing job ID: 12370144568
Job result: FAILURE

@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/sca

Testing job ID: 12370220540
Job result: FAILURE
Testing job ID: 12370283532
Job result: FAILURE
Testing job ID: 12370283532
Job result: FAILURE

@horiagunica horiagunica changed the title fix(pre_commit): Bumped pre_commit_terraform and pre_commit versions fix(pre_commit): DO NOT MERGE - Bumped pre_commit_terraform and pre_commit versions Dec 17, 2024
@horiagunica
Copy link
Contributor Author

horiagunica commented Dec 17, 2024 •
edited by github-actions bot
Loading

/sca

Testing job ID: 12372528240
Job result: FAILURE

@horiagunica
Copy link
Contributor Author

Closing PR - was used for remote CI workflow testing.

@horiagunica horiagunica deleted the pre_commit_fmt_fix branch December 17, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@horiagunica