8000 Fixes wrongly appended "v" to the appVersion by Ilyesbdlala · Pull Request #819 · secureCodeBox/secureCodeBox · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@Ilyesbdlala
Copy link
Member
@Ilyesbdlala Ilyesbdlala commented Nov 16, 2021

closes #817

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.
  • Make codeclimate checks happy

i.e v1.0 or 1.0
Added on push trigger for testing
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala Ilyesbdlala added the bug Bugs label Nov 16, 2021
@Ilyesbdlala Ilyesbdlala added this to the v3.4.0 milestone Nov 16, 2021
@Ilyesbdlala Ilyesbdlala self-assigned this Nov 16, 2021
@Ilyesbdlala Ilyesbdlala changed the title SCB bot now makes sure the release and local version have same format Fixes wrongly appended "v" to the appVersion Nov 16, 2021
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
merging to allow for accurate version check from branch

Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala Ilyesbdlala marked this pull request as ready for review November 16, 2021 14:55
this avoids issues that occur when curl request fails

Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@J12934 J12934 merged commit ab859dd into main Nov 17, 2021
@J12934 J12934 deleted the bugfix/scb-bot-version-control branch November 17, 2021 09:18
Ilyesbdlala added a commit that referenced this pull request Nov 17, 2021
This is done to get it inline with the #819 PR
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SCB-BOT] Wrongly appended "v" to the appVersion

3 participants

0