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

Technical Feature/NR-61 Semantic versioning #191

Merged

Conversation

stas-at-ibm
Copy link
Collaborator

@stas-at-ibm stas-at-ibm commented Nov 12, 2023

Closes #190.
Closes #174.

@stas-at-ibm stas-at-ibm added the technical feature Improvement or change on how the application implements its features label Nov 12, 2023
@stas-at-ibm stas-at-ibm added this to the Minimum Viable Product milestone Nov 12, 2023
@stas-at-ibm stas-at-ibm self-assigned this Nov 12, 2023
@stas-at-ibm stas-at-ibm changed the title Technical Feature: NR-61 Semantic versioning Technical Feature/NR-61 Semantic versioning Nov 12, 2023
@stas-at-ibm stas-at-ibm marked this pull request as ready for review November 18, 2023 11:49
@stas-at-ibm stas-at-ibm force-pushed the technical-feature/NR-61-introduce-semantic-versioning branch 4 times, most recently from 74d9383 to f8b2134 Compare November 24, 2023 09:29
Removed old documentation and added project worklow docs with
trunk based development, squash and a semantic versioning guides.
Added first version to package*.json and git tag to latest commit.
Added monorepo version pattern.
Added pipeline step for validating git tag.
Added push and deploy conditions for master.
Added taggin of docker images.

Issue-Ref: #190
@stas-at-ibm stas-at-ibm force-pushed the technical-feature/NR-61-introduce-semantic-versioning branch from f8b2134 to e2e6352 Compare November 25, 2023 14:12
@stas-at-ibm stas-at-ibm merged commit a67557a into master Nov 25, 2023
4 checks passed
@stas-at-ibm stas-at-ibm deleted the technical-feature/NR-61-introduce-semantic-versioning branch December 19, 2023 18:38
stas-at-ibm pushed a commit that referenced this pull request Dec 25, 2023
cicd-backend.yml is deprecated and was removed in #191.

Issue-Ref: #32
stas-at-ibm added a commit that referenced this pull request Dec 25, 2023
Added a new aggregation method which does not perform any diff
calculations on recieved and stored steam apps. Instead the database
is taking care of that. It stores only new steam apps ignoring the
existing ones.

This should lower the memory consumption and prevent the application
from crashing on Render where its running at the time of this
writing.

The method is collectSteamAppsDiffOnDbLayer and is used via a
simple environment variable feature toggle.

Changed the tests for the old and the new method to 
given-when-then style in descriptions and removed tests which knew
to much about the implementation details.

By mistake cicd-backend.yml was in master but it is deprecated and
was removed in #191. Removed it again.

The cloud db steam_apps collection had many duplications which are
cleaned up now. Also a database management documentation was
added.

Issue-Ref: #32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical feature Improvement or change on how the application implements its features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Introduce semantic versioning Add version tags to backend and to images.
2 participants