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.

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

Enable Integration tests procedure and add CONTRIBUTING MD file #59

Merged
merged 3 commits into from
Apr 17, 2024

Conversation

nirarg
Copy link
Contributor

@nirarg nirarg commented Apr 7, 2024

No description provided.

@nirarg
Copy link
Contributor Author

nirarg commented Apr 7, 2024

/cc @p3ck

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@nirarg nirarg force-pushed the integration branch 2 times, most recently from 378a82b to 92268de Compare April 10, 2024 06:25
CONTRIBUTING.md Outdated

> **_NOTE:_** Make sure to replace `<working directory>` with the actual path where you want to clone the repositories and set up your environment.

1. **Clone This Repository**:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
1. **Clone This Repository**:
1. **Clone Azure Roles Collection Repository**:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed the descriptions in the clone steps.
Please review

@anna-savina
Copy link
Collaborator

lgtm
Thank you!

@prabinovRedhat
Copy link
Collaborator

lgtm, all tests passed

Copy link
Contributor

@p3ck p3ck left a comment

Choose a reason for hiding this comment

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

lgtm

@nirarg nirarg force-pushed the integration branch 2 times, most recently from d0918aa to 55b3794 Compare April 16, 2024 11:49
@p3ck p3ck self-requested a review April 16, 2024 11:56
@p3ck
Copy link
Contributor

p3ck commented Apr 16, 2024

Looks like the changelog format isn't quite right..

ERROR: Found 1 changelog issue(s) which need to be resolved: ERROR: changelogs/fragments/20240416-integration.yml:0:0: section "bugfixes" list items must be type str not dict

@nirarg
Copy link
Contributor Author

nirarg commented Apr 16, 2024

Looks like the changelog format isn't quite right..

ERROR: Found 1 changelog issue(s) which need to be resolved: ERROR: changelogs/fragments/20240416-integration.yml:0:0: section "bugfixes" list items must be type str not dict

@p3ck fixed this error, I think it ready for merge now
@alinabuzachis can you please review it again?

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
nirarg added 3 commits April 17, 2024 10:54
These error were detected by the integration test:
* "azure_manage_security_group_region" should be optional, as it not
  required when the \"Resource Group\" is already exists.
* Fix "purge_rules" and "rules_to_remove" indentation in the arguments spec
@p3ck p3ck merged commit 0850c5e into redhat-cop:main Apr 17, 2024
18 checks passed
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.

5 participants