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

Added contribute document #54

Merged
merged 3 commits into from
Aug 4, 2023

Conversation

kiranchavala
Copy link
Collaborator

No description provided.

CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: Harikrishna <[email protected]>
Copy link
Contributor

@harikrishna-patnala harikrishna-patnala left a comment

Choose a reason for hiding this comment

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

LGTM

@kiranchavala
Copy link
Collaborator Author

@rohityadavcloud @borisstoyanov @shwstppr

Could you please review this

Copy link

@shwstppr shwstppr left a comment

Choose a reason for hiding this comment

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

@kiranchavala there are some references to CloudStack repo and confluence. Are they correct?

CONTRIBUTING.md Outdated

To initiate the pull request, do the following:

1. In your browser, navigate to your forked repository: [https://github.com/YOUR_ACCOUNT/cloudstack](https://github.com/YOUR_ACCOUNT/cloudstack)

Choose a reason for hiding this comment

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

Is it correct to point to CloudStack repo?

CONTRIBUTING.md Outdated Show resolved Hide resolved
@rohityadavcloud rohityadavcloud merged commit eaf06e9 into apache:main Aug 4, 2023
1 check passed
@harikrishna-patnala harikrishna-patnala added this to the v0.5.0 milestone Aug 7, 2023
poddm pushed a commit to poddm/cloudstack-terraform-provider that referenced this pull request Oct 31, 2023
* Added contribute document

* Update CONTRIBUTING.md

Co-authored-by: Harikrishna <[email protected]>

* Update CONTRIBUTING.md

---------

Co-authored-by: Harikrishna <[email protected]>
Co-authored-by: Rohit Yadav <[email protected]>
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.

4 participants