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

updated github.go to check /api/v3 | issue> #1980 #2546

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

varshneydevansh
Copy link
Contributor

Issue> Validate --github-api-url #1980

Sorry again, creating the PR for this issue as I also added the update for the other file of other issue with the same branch which got added with this PR. I was trying to create a new PR for the other issue, but eventually ended-up discarding all the changes. When I saw the PR it was showing closed.

I added the check for the "/api/v3" for the github-api-url to make sure we remain consistent with the link.

Thanks, Ali Khajeh and Alistair Scott for helping and guiding with my second open-source contribution.

Copy link
Contributor Author

@varshneydevansh varshneydevansh left a comment

Choose a reason for hiding this comment

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

I apologize Alistair. The code is now correctly added.

@aliscott aliscott self-requested a review July 6, 2023 13:31
Copy link
Member

@aliscott aliscott left a comment

Choose a reason for hiding this comment

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

Thanks @varshneydevansh. Test failures look unrelated so this is good to go.

@aliscott aliscott merged commit b086f03 into infracost:master Jul 6, 2023
9 of 10 checks passed
@varshneydevansh
Copy link
Contributor Author

Thank you so much, @aliscott
This is my first PR merge, looking to contribute more. (•‿•)

@varshneydevansh varshneydevansh deleted the branch-api-v3 branch July 6, 2023 16:32
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.

None yet

2 participants