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

Prepare for next development iteration - 0.279-SNAPSHOT #18552

Merged
merged 2 commits into from
Nov 2, 2022

Conversation

g1y
Copy link
Member

@g1y g1y commented Oct 24, 2022

No description provided.

@g1y g1y requested review from a team, vinothchandar and 7c00 as code owners October 24, 2022 16:58
@g1y g1y requested a review from presto-oss October 24, 2022 16:58
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 24, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: g1y / name: Guy ☀️ Moore (2c7973c)
  • ✅ login: presto-release-bot (59260b0)

@g1y g1y force-pushed the next-snapshot-0.279-SNAPSHOT branch 2 times, most recently from 7d3273e to fdf3530 Compare October 25, 2022 16:39
g1y and others added 2 commits October 25, 2022 17:11
The new release process for presto creates pull requests
that have an incremented snapshot which hasn't
been built by anything yet. There are tests that are building
test coverage but skipping presto server. This creates a problem
where they are requesting dependencies that aren't built yet.

We don't need test-coverage in these tests, so it's best if
we just skip building those modules.
@g1y g1y force-pushed the next-snapshot-0.279-SNAPSHOT branch from fdf3530 to 59260b0 Compare October 26, 2022 00:12
@jarias-lfx
Copy link

/easycla

@facebook-github-bot
Copy link
Collaborator

@g1y has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@g1y g1y merged commit 0a7c1cf into master Nov 2, 2022
Copy link

@lemurcina lemurcina left a comment

Choose a reason for hiding this comment

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

Need to be checked.

@wanglinsong wanglinsong mentioned this pull request Jan 12, 2023
30 tasks
@wanglinsong wanglinsong deleted the next-snapshot-0.279-SNAPSHOT branch April 13, 2023 18:45
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.

6 participants