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.

Sign up for GitHub

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

Jump to bottom

arcticdb 4.5.0 #277

Merged
merged 8 commits into from
Aug 15, 2024
Merged

arcticdb 4.5.0 #277

merged 8 commits into from
Aug 15, 2024

Conversation

G-D-Petrov
Copy link
Contributor

@G-D-Petrov G-D-Petrov commented Aug 13, 2024 •
edited by jjerphan
Loading

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@G-D-Petrov
Copy link
Contributor Author

@conda-forge-admin, please rerender

@conda-forge-webservices
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

conda-forge-webservices[bot] and others added 2 commits August 13, 2024 12:52
@G-D-Petrov
Copy link
Contributor Author

@conda-forge-admin, please rerender

@jjerphan jjerphan changed the title Release v4.5.0 arcticdb 4.5.0 Aug 13, 2024
jjerphan
jjerphan approved these changes Aug 13, 2024
View reviewed changes
Copy link
Member

@jjerphan jjerphan left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM given 🟢 CI checks.

Need not be addressed: a couple of doc changes and comments could be ported from rc to main, see the absolute diff between the two branches.

DrNickClarke
DrNickClarke approved these changes Aug 13, 2024
View reviewed changes
Copy link

@DrNickClarke DrNickClarke left a comment

Choose a reason for hiding this comment

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

LGTM

@G-D-Petrov
Copy link
Contributor Author

@conda-forge-admin, please rerender

@jjerphan jjerphan mentioned this pull request Aug 14, 2024
Closed
3 tasks
@G-D-Petrov
Exclude failing tests
a25cc48 
Increase build number

Revert "Increase build number"

This reverts commit e84ec25a89ec3d43e21980b13475e8c0eeb18690.
@G-D-Petrov
Copy link
Contributor Author

@conda-forge-admin, please rerender

@github-actions GitHub Actions
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/arcticdb-feedstock/actions/runs/10387216477.

jjerphan and others added 2 commits August 14, 2024 15:18
@jjerphan
Retrigger CI
fc5aa3d 
Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@G-D-Petrov
Exclude flaky test
4161cfd
@G-D-Petrov
Copy link
Contributor Author

@conda-forge-admin, please rerender

@github-actions GitHub Actions
Copy link
Contributor

Hi! This is the friendly automated conda-forge-webservice.

I tried to rerender for you, but it looks like there was nothing to do.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/arcticdb-feedstock/actions/runs/10401551049.

@G-D-Petrov G-D-Petrov merged commit e8534d0 into conda-forge:main Aug 15, 2024
12 checks passed
@G-D-Petrov G-D-Petrov deleted the 450 branch August 15, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjerphan jjerphan jjerphan approved these changes

@DrNickClarke DrNickClarke DrNickClarke approved these changes

@Alex-PLACET Alex-PLACET Awaiting requested review from Alex-PLACET Alex-PLACET is a code owner

@DerThorsten DerThorsten Awaiting requested review from DerThorsten DerThorsten is a code owner

@Hind-M Hind-M Awaiting requested review from Hind-M Hind-M is a code owner

@JohanMabille JohanMabille Awaiting requested review from JohanMabille JohanMabille is a code owner

@Klaim Klaim Awaiting requested review from Klaim Klaim is a code owner

@alexowens90 alexowens90 Awaiting requested review from alexowens90 alexowens90 is a code owner

@jamesmunro jamesmunro Awaiting requested review from jamesmunro jamesmunro is a code owner

@joe-iddon joe-iddon Awaiting requested review from joe-iddon joe-iddon is a code owner

@mehertz mehertz Awaiting requested review from mehertz mehertz is a code owner

@muhammadhamzasajjad muhammadhamzasajjad Awaiting requested review from muhammadhamzasajjad muhammadhamzasajjad is a code owner

@phoebusm phoebusm Awaiting requested review from phoebusm phoebusm is a code owner

@poodlewars poodlewars Awaiting requested review from poodlewars poodlewars is a code owner

@qc00 qc00 Awaiting requested review from qc00 qc00 is a code owner

@vasil-pashov vasil-pashov Awaiting requested review from vasil-pashov vasil-pashov is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@G-D-Petrov @jjerphan @DrNickClarke