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

add test to check if temporal spec can flatten timestamps to obtain outcome timestamps #292

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

bokajgd
Copy link
Contributor

@bokajgd bokajgd commented Oct 4, 2023

Fixes #290

@github-actions
Copy link
Contributor

github-actions bot commented Oct 4, 2023

Looks like some formatting rules failed.

✨ The action has attempted automatic fixes ✨

If any were succesful, they were committed to the branch.
We suggest using git pull --rebase to apply them locally.

If some errors could not be fixed automatically, you can:

🏎️ Get results locally by running pre-commit run --all-files
🕵️ Examine the results in the Run pre-commit section of this workflow pre-commit

We also strongly recommend setting up the ruff and black extensions to auto-format on save in your chosen editor.

@bokajgd bokajgd requested a review from frillecode October 4, 2023 12:31
@KennethEnevoldsen
Copy link
Contributor

Error fixed in #299, will merge

@KennethEnevoldsen KennethEnevoldsen merged commit e4812b1 into main Oct 10, 2023
11 of 12 checks passed
@KennethEnevoldsen KennethEnevoldsen deleted the jakdam/test_timestamps_as_values branch October 10, 2023 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add func for keeping outcome timestamp col
3 participants