Skip to content

Refactor 1#44

Merged
thisisibrahimd merged 3 commits intomasterfrom
refactor-1
Apr 6, 2022
Merged

Refactor 1#44
thisisibrahimd merged 3 commits intomasterfrom
refactor-1

Conversation

@thisisibrahimd
Copy link
Copy Markdown
Member

Implemented mono repo and added scoretrak-scoretrakapis repo contained buf generated output.

we are using yarn to install and build in the Dockerfile so it is best to make it consistent.
removed package-lock.json as well.
as well as add context to github action docker image build job.
@thisisibrahimd thisisibrahimd added the enhancement New feature or request label Apr 6, 2022
@thisisibrahimd thisisibrahimd self-assigned this Apr 6, 2022
@thisisibrahimd thisisibrahimd linked an issue Apr 6, 2022 that may be closed by this pull request
@thisisibrahimd thisisibrahimd marked this pull request as ready for review April 6, 2022 17:32
@thisisibrahimd
Copy link
Copy Markdown
Member Author

Going to test ci with pre-release

@thisisibrahimd
Copy link
Copy Markdown
Member Author

CI is as usual (https://github.com/ScoreTrak/client/actions/runs/2104081944) and inspecting the container with dive (https://gochronicles.com/dive/) show the same output as previous containers.

@L1ghtman2k
Copy link
Copy Markdown
Member

This is a lot of files to check haha, If it looked fine on ur end, let's merge this

@thisisibrahimd
Copy link
Copy Markdown
Member Author

Yeah a lot of files, haha. Just moving them from one folder to another spams the files changed count :(.

@thisisibrahimd thisisibrahimd merged commit d2bcbfd into master Apr 6, 2022
@thisisibrahimd thisisibrahimd deleted the refactor-1 branch April 16, 2022 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor

2 participants