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

Fix OrgsUpdater identifying new releases as old #516

Merged
merged 3 commits into from
May 10, 2021

Conversation

aly76
Copy link
Contributor

@aly76 aly76 commented May 9, 2021

  • Pass the releaseWithMatchingHashId to OrgsUpdater, instead of determining itself
    whether is new release

  • Fix branch flag on changelog:generate so that it's not required

Pass the releaseWithMatchingHashId to OrgsUpdater instead of determining itself
whether is new release

Fix branch flag on changelog:generate so that it's not required
@aly76 aly76 added the bug Something isn't working label May 9, 2021
@aly76 aly76 requested a review from azlam-abdulsalam May 9, 2021 11:44
@codecov-commenter
Copy link

codecov-commenter commented May 9, 2021

Codecov Report

Merging #516 (54d99b8) into develop (a91a531) will decrease coverage by 0.07%.
The diff coverage is 80.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #516      +/-   ##
===========================================
- Coverage    79.47%   79.39%   -0.08%     
===========================================
  Files           38       38              
  Lines         1296     1291       -5     
  Branches       223      222       -1     
===========================================
- Hits          1030     1025       -5     
  Misses         266      266              
Impacted Files Coverage Δ
...powerscripts-cli/src/impl/changelog/OrgsUpdater.ts 87.50% <80.00%> (-1.39%) ⬇️

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 0cc4988...54d99b8. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants