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 X windows part 12/N #27512

Merged
merged 12 commits into from
Nov 1, 2022
Merged

Conversation

Neumann-A
Copy link
Contributor

No description provided.

github-actions[bot]
github-actions bot previously approved these changes Oct 27, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You have modified or added at least one vcpkg.json where you should check the license field.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libdmx/vcpkg.json
  • ports/xbitmaps/vcpkg.json

Valid values for the license field can be found in the documentation

@Neumann-A Neumann-A marked this pull request as draft October 27, 2022 22:31
@LilyWangLL LilyWangLL added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Oct 28, 2022
github-actions[bot]
github-actions bot previously approved these changes Oct 28, 2022
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

You have modified or added at least one vcpkg.json where you should check the license field.

If you feel able to do so, please consider adding a "license" field to the following files:

  • ports/libdmx/vcpkg.json
  • ports/xbitmaps/vcpkg.json

Valid values for the license field can be found in the documentation

github-actions[bot]
github-actions bot previously approved these changes Oct 28, 2022
@Neumann-A Neumann-A marked this pull request as ready for review October 28, 2022 22:36
LilyWangLL
LilyWangLL previously approved these changes Oct 31, 2022
Cheney-W
Cheney-W previously approved these changes Oct 31, 2022
@Cheney-W Cheney-W added the info:reviewed Pull Request changes follow basic guidelines label Oct 31, 2022
Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

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

New port checklist:

  • Review the code ✅✅✅ ⚠️
  • Check the name against https://repology.org/ ✅✅✅
  • Check the name against Bing/Google. ✅ Skipped.
  • Check the source code for optional find_packages ✅ None are CMake.
  • Check that the versioning scheme matches what upstream says ✅✅✅
  • Check that the license in vcpkg.json matches what upstream says ✅✅✅
  • Check that the license installed by the port matches what upstream says ✅✅✅
  • Check that the source code comes from the upstream project's authoritative source. ✅✅✅
  • Check that the generated usage is accurate. ✅✅✅ No usage.
  • Check for issues that are resolved. ✅✅✅

I pushed a SHA fix and a copyright install style fix.

scripts/ci.baseline.txt Show resolved Hide resolved
@Cheney-W Cheney-W added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines requires:author-response labels Nov 1, 2022
@BillyONeal BillyONeal merged commit 1f825db into microsoft:master Nov 1, 2022
@Neumann-A Neumann-A deleted the x_windows_part12 branch November 1, 2022 22:19
@Cheney-W Cheney-W added the info:reviewed Pull Request changes follow basic guidelines label Nov 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants