Skip to content
This repository has been archived by the owner on Oct 16, 2024. It is now read-only.

Automated testing #42

Closed
wants to merge 42 commits into from
Closed

Automated testing #42

wants to merge 42 commits into from

Conversation

vlorinc
Copy link
Contributor

@vlorinc vlorinc commented Nov 29, 2023

Added automated tests of grpc client functions.

@github-actions github-actions bot added size/XXL enhancement New feature or request labels Nov 29, 2023
vlorinc and others added 17 commits November 29, 2023 15:35
Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.2 to 1.5.3.
- [Release notes](https://github.com/golang/protobuf/releases)
- [Commits](golang/protobuf@v1.5.2...v1.5.3)

---
updated-dependencies:
- dependency-name: github.com/golang/protobuf
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
- Add tool fetching in Makefile for addlicense, goimports and
  go-proto-gen tools
- Regenerate proto files with new tools versions
dependabot bot and others added 14 commits December 12, 2023 12:12
Bumps google.golang.org/protobuf from 1.28.1 to 1.31.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 4 to 5.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](actions/setup-go@v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.53.0 to 1.59.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.53.0...v1.59.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@vlorinc vlorinc marked this pull request as ready for review December 12, 2023 11:26
@vlorinc vlorinc requested a review from guvenc December 12, 2023 11:27
@vlorinc vlorinc self-assigned this Dec 12, 2023
Copy link
Collaborator

@guvenc guvenc left a comment

Choose a reason for hiding this comment

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

@vlorinc
This PR has 42 commits. Looks like you merged the main branch to development branch.
Please create a new PR branch. Dont change this one but a create new branch based on main and just cherry pick the commits relevant to this new change on that new PR branch and open a new PR.

@@ -1,7 +1,7 @@
name: test

on:
pull_request:
pull_request_target:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use pull_request and not "pull_request_target".
This is a change we did during the immigration.

@guvenc
Copy link
Collaborator

guvenc commented Dec 14, 2023

Changes in this PR are merged with another PR #52

@guvenc guvenc closed this Dec 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants