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

feat: Add Missing Methods for Execution details #28

Merged
merged 3 commits into from
Oct 20, 2023

Conversation

sccalabr
Copy link
Contributor

@sccalabr sccalabr commented Oct 10, 2023

Resolves #18

@sccalabr sccalabr changed the title Add Missing Methods for Execution details feat: Add Missing Methods for Execution details Oct 10, 2023
@doomspork
Copy link
Collaborator

@sccalabr could you please address the failing CI checks?

@sccalabr sccalabr marked this pull request as draft October 11, 2023 01:16
@sccalabr sccalabr force-pushed the issue-18 branch 2 times, most recently from 938408b to 5da7d85 Compare October 11, 2023 01:28
@sccalabr sccalabr marked this pull request as ready for review October 11, 2023 01:29
@sccalabr sccalabr marked this pull request as draft October 11, 2023 01:29
@sccalabr
Copy link
Contributor Author

do work flows not run on drafts?

@sccalabr sccalabr marked this pull request as ready for review October 11, 2023 03:57
@unicodeveloper
Copy link

@sccalabr Please attend to the failing tests.

@sccalabr sccalabr marked this pull request as draft October 11, 2023 15:38
@sccalabr sccalabr marked this pull request as ready for review October 12, 2023 04:51
@sccalabr sccalabr marked this pull request as draft October 12, 2023 04:52
@sccalabr sccalabr marked this pull request as ready for review October 14, 2023 18:22
@sccalabr
Copy link
Contributor Author

any problems with the pr?

test/novu/execution_details_test.exs Outdated Show resolved Hide resolved
Copy link
Contributor Author

@sccalabr sccalabr left a comment

Choose a reason for hiding this comment

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

addressing comments

@unicodeveloper
Copy link

@sccalabr All the CI tests are failing. Please look into it

@btkostner
Copy link
Collaborator

btkostner commented Oct 19, 2023

The CI failures look to be coming from main branch. I'll open a PR soon to fix them

Update: #32

Copy link
Collaborator

@btkostner btkostner left a comment

Choose a reason for hiding this comment

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

Code looks good. Just need to fix the format error in CI by running mix format and we should be good.

Removing unused variable
@doomspork doomspork dismissed btkostner’s stale review October 20, 2023 13:11

Code has been formatted

@doomspork doomspork merged commit aa28c83 into novuhq:main Oct 20, 2023
9 checks passed
@scopsy scopsy mentioned this pull request Oct 20, 2023
doomspork pushed a commit that referenced this pull request Nov 9, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.3.0](v0.2.0...v0.3.0)
(2023-10-26)


### Features

* add environments module
([#24](#24))
([11487c6](11487c6))
* add Idempotency-Key header and exponential retry mechanism
([#31](#31))
([fd3754a](fd3754a))
* Add Missing Methods for Execution details
([#28](#28))
([aa28c83](aa28c83)),
closes [#18](#18)
* Add Missing Methods for Messages
([#29](#29))
([f718e2a](f718e2a)),
closes [#20](#20)
* set default maximum retry to zero
([#35](#35))
([c7bd600](c7bd600)),
closes [#33](#33)

---
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Missing Methods for Execution details
4 participants