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

Fix test and add changelog for 0.7.0 #84

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

rossf7
Copy link
Contributor

@rossf7 rossf7 commented Jun 11, 2024

No description provided.

@rossf7 rossf7 requested a review from mrchrisadams June 11, 2024 12:04
rossf7 added 2 commits June 11, 2024 14:26
@@ -1,24 +1,27 @@
name: docker-integration-test
on: [push]
on: [pull_request, push]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mrchrisadams Push doesn't work for forks 🤦‍♂️ now we will run the integration tests once a PR is created.

I also did some code gardening and updated the actions versions.

- Update Electricity Maps provider to return both estimated and actual values.
Note: There is a delay before actual values are available.
- Update location parameter to support multiple locations separated with a
comma.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mrchrisadams PTAL 🙏 Do these release notes look OK?

uses: actions/checkout@v4
-
name: Set up Go
uses: actions/setup-go@v5
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The go version in the Actions VM is usually pretty old. We used setup-go in the release action. Now we use it in all of them

Copy link
Member

@mrchrisadams mrchrisadams left a comment

Choose a reason for hiding this comment

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

LGTM @rossf7, thank you. Merge away.

@rossf7 rossf7 merged commit 3d2b9ac into thegreenwebfoundation:main Jun 11, 2024
3 checks passed
@rossf7 rossf7 deleted the fix/tests-and-changelog branch June 11, 2024 15:49
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