ci: add diff_check workflow

[burntcarrot]

Added a generate diff check. Related #525.

[codecov]

Codecov Report

Merging #664 (7b18d6f) into master (96ab6c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #664   +/-   ##
=======================================
  Coverage   17.57%   17.57%           
=======================================
  Files          19       19           
  Lines         768      768           
=======================================
  Hits          135      135           
  Misses        626      626           
  Partials        7        7           

Flag	Coverage Δ
unittests	17.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 96ab6c0...7b18d6f. Read the comment docs.

[Xuanwo]

Sorry, I think we had some communication problems. I'm sorry I didn't read your example carefully in #525. We need to add diff_check for every single service instead of in go-storage. (So that we can check the diff while there are new push / PRs for services)

[burntcarrot]

My guess is that I've not added the relative paths to each Makefile for each service, since you have to be in a directory which contains a Makefile.

For example, the paths should be pointing to their respective services like /home/runner/work/go-storage/go-storage/go-service-azblob and not /home/runner/work/go-storage/go-storage.

Edited wrt. to the comment above.

[Xuanwo]

We need to add a diff-check.yml under https://github.com/beyondstorage/go-service-s3/tree/master/.github/workflows like what we do for gofmt.yml. So we can run the check every time there is a new push or PR for go-service-s3.

GitHub

beyondstorage/go-service-s3

Amazon S3 support for go-storage. Contribute to beyondstorage/go-service-s3 development by creating an account on GitHub.

[burntcarrot]

Sorry, I think we had some communication problems. I'm sorry I didn't read your example carefully in #525. We need to add diff_check for every single service instead of in go-storage. (So that we can check the diff while there are new push / PRs for services)

I was suspecting this earlier. Since generated.go lies in the repos of each service, hence, the actions should be run there.

Anyways, thanks for clarifying now! I'll make PRs for each service.

[Xuanwo]

Closed as discussed in #664 (comment)