Skip to content

Fix the package logic, and add windows build python logic #250

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

Merged
merged 44 commits into from
Apr 29, 2022

Conversation

cynthiajoan
Copy link
Contributor

Description

Provide details of the change, and generalize the change in the PR title above.


Testing

Describe how you've tested these changes.


Type of Change

Place an x the applicable box:

  • Bug fix. Add the issue # below if applicable.
  • New feature. A non-breaking change which adds functionality.
  • Other, such as a build process or documentation change.

@cynthiajoan cynthiajoan changed the title Windows fix Fix the package logic, and add windows build python logic Apr 29, 2022
@cynthiajoan cynthiajoan requested a review from a-maurice April 29, 2022 01:56
description: 'run id for sdk_build.yml'
required: false
release_label:
description: 'If the package is intened to run for a release, put <version>-RC## for label'
Copy link
Collaborator

Choose a reason for hiding this comment

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

intended

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -21,7 +21,8 @@ gcs_key_file.json
*_build/
*_unity/
*_unity_seperate/
*_unity_separate/
output/
Copy link
Collaborator

Choose a reason for hiding this comment

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

The other separate was intentional (different spellings, with the a one being the correct one)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done, was confused a lot, now totally make sense...

@cynthiajoan cynthiajoan merged commit fd39689 into main Apr 29, 2022
@github-actions github-actions bot added the tests: in-progress This PR's integration tests are in progress. label Apr 29, 2022
@github-actions
Copy link

github-actions bot commented Apr 29, 2022

❌  Integration test FAILED

Requested by @cynthiajoan on commit fd39689
Last updated: Fri Apr 29 13:18 PDT 2022
View integration test log & download artifacts

Failures Configs
missing_log [BUILD] [ERROR] [2019] [macos] [1/3 Platform(s): iOS]
firestore [TEST] [FLAKINESS] [2019] [macos] [1/4 Platform(s): Android] [android_target]
functions [TEST] [ERROR] [2019] [macos] [1/4 Platform(s): windows]
messaging [TEST] [ERROR] [2019] [macos] [1/4 Platform(s): Android] [android_target]
storage [BUILD] [ERROR] [2019] [macos] [2/3 Platform(s): Windows,macOS,Linux Android]

@github-actions github-actions bot added the tests: failed This PR's integration tests failed. label Apr 29, 2022
@cynthiajoan cynthiajoan deleted the prepare_tgz branch April 29, 2022 20:09
@firebase-workflow-trigger firebase-workflow-trigger bot removed the tests: in-progress This PR's integration tests are in progress. label Apr 29, 2022
@firebase firebase locked and limited conversation to collaborators Jun 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tests: failed This PR's integration tests failed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants