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

fix: moves sample file out of sources folder #439

Conversation

GevaZeichner
Copy link

@GevaZeichner GevaZeichner commented Jan 12, 2024

This prevented the package from being used in Bazel/rules_swift_package_manager.

Error:
rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios/BUILD.bazel:19:14: in srcs attribute of swift_library rule @rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios//:Sources_Rudder: source file '@rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios//:Sources/Classes/Networking/APIClient/SampleBatchPayload.json' is misplaced here (expected swift)

This prevented the package from being used in Bazel: https://github.com/cgrindel/rules_swift_package_manager

Error:
`rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios/BUILD.bazel:19:14: in srcs attribute of swift_library rule @rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios//:Sources_Rudder: source file '@rules_swift_package_manager~0.25.0~swift_deps~swiftpkg_rudder_sdk_ios//:Sources/Classes/Networking/APIClient/SampleBatchPayload.json' is misplaced here (expected swift)`
pallabmaiti
pallabmaiti previously approved these changes Jan 30, 2024
@desusai7 desusai7 changed the base branch from master-v2 to develop-v2 January 30, 2024 09:50
@desusai7 desusai7 dismissed pallabmaiti’s stale review January 30, 2024 09:50

The base branch was changed.

@desusai7 desusai7 requested a review from pallabmaiti January 30, 2024 09:50
pallabmaiti
pallabmaiti previously approved these changes Jan 30, 2024
@pallabmaiti pallabmaiti self-requested a review January 30, 2024 10:05
@pallabmaiti
Copy link
Contributor

@GevaZeichner please raise the PR against develop-v2.

@pallabmaiti pallabmaiti changed the base branch from develop-v2 to master-v2 January 30, 2024 10:07
@pallabmaiti pallabmaiti dismissed their stale review January 30, 2024 10:07

The base branch was changed.

@pallabmaiti pallabmaiti added the v2 label Feb 14, 2024
@desusai7 desusai7 changed the base branch from master-v2 to develop-v2 March 28, 2024 11:27
@desusai7 desusai7 requested a review from a team as a code owner March 28, 2024 11:27
@desusai7
Copy link
Contributor

Hi @GevaZeichner,

We are closing this PR, as we are tackling this part of another PR internally.

@desusai7 desusai7 closed this Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants