Skip to content

fix: Fix Bill without Feb#13487

Merged
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
easayliu:main
Aug 31, 2023
Merged

fix: Fix Bill without Feb#13487
kodiakhq[bot] merged 2 commits intocloudquery:mainfrom
easayliu:main

Conversation

@easayliu
Copy link
Copy Markdown
Contributor

@easayliu easayliu commented Aug 31, 2023

Fix HistoryBillingCycles. when data is 28~31,month without 02 or another without 31 month.

BEGIN_COMMIT_OVERRIDE
fix: Properly account for months of different length in HistoryBillingCycles (#13487)

END_COMMIT_OVERRIDE

@easayliu easayliu requested review from a team and hydratim and removed request for a team August 31, 2023 07:51
@erezrokah erezrokah requested review from a team and removed request for hydratim August 31, 2023 07:52
@easayliu easayliu changed the title fix(alicloud)!: Fix Bill without Feb。 fix(main): alicloud source ,Fix Bill without Feb。 Aug 31, 2023
@erezrokah erezrokah changed the title fix(main): alicloud source ,Fix Bill without Feb。 fix!: Fix Bill without Feb Aug 31, 2023
Copy link
Copy Markdown
Contributor

@candiduslynx candiduslynx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @easayliu!

I think what you want to achieve can be simplified.
See https://go.dev/play/p/TvNVFcYyQdC

Co-authored-by: Alex Shcherbakov <candiduslynx@users.noreply.github.com>
@candiduslynx candiduslynx added the automerge Automatically merge once required checks pass label Aug 31, 2023
@kodiakhq kodiakhq bot merged commit 56d0e45 into cloudquery:main Aug 31, 2023
@candiduslynx candiduslynx changed the title fix!: Fix Bill without Feb fix: Fix Bill without Feb Aug 31, 2023
kodiakhq bot pushed a commit that referenced this pull request Aug 31, 2023
🤖 I have created a release *beep* *boop*
---


## [4.0.7](plugins-source-alicloud-v4.0.6...plugins-source-alicloud-v4.0.7) (2023-08-31)


### Bug Fixes

* Properly account for months of different length in `HistoryBillingCycles` ([#13487](#13487)) ([56d0e45](56d0e45))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants