Skip to content

Publish script argument uses the wrong file path separator#524

Merged
tom-un merged 1 commit intomicrosoft:masterfrom
mganandraj:MasterPublishDroidFix
Jul 29, 2020
Merged

Publish script argument uses the wrong file path separator#524
tom-un merged 1 commit intomicrosoft:masterfrom
mganandraj:MasterPublishDroidFix

Conversation

@mganandraj
Copy link
Copy Markdown

@mganandraj mganandraj commented Jul 29, 2020 •
edited by ghost
Loading

Fixing publish script as the task argument uses wrong file path separator to be used on Linux/Ubuntu host

Please select one of the following

  • I am removing an existing difference between facebook/react-native and microsoft/react-native-macos 👍
  • I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos 👍
  • I am making a fix / change for the macOS implementation of react-native
  • I am making a change required for Microsoft usage of react-native

No code change

Summary

Fixing publish script as the task argument uses wrong file path separator to be used on Linux/Ubuntu host

Changelog

Fixing publish script as the task argument uses wrong file path separator to be used on Linux/Ubuntu host

Microsoft Reviewers: Open in CodeFlow

@mganandraj mganandraj requested a review from tom-un as a code owner July 29, 2020 00:27
@pull-bot
Copy link
Copy Markdown

pull-bot commented Jul 29, 2020

Messages
📖

📋 Verify Changelog Format - A changelog entry has the following format: [CATEGORY] [TYPE] - Message.

DetailsCATEGORY may be:
  • General
  • macOS
  • iOS
  • Android
  • JavaScript
  • Internal (for changes that do not need to be called out in the release notes)

TYPE may be:

  • Added, for new features.
  • Changed, for changes in existing functionality.
  • Deprecated, for soon-to-be removed features.
  • Removed, for now removed features.
  • Fixed, for any bug fixes.
  • Security, in case of vulnerabilities.

MESSAGE may answer "what and why" on a feature level. Use this to briefly tell React Native users about notable changes.
📖 📋 Missing Test Plan - Can you add a Test Plan? To do so, add a "## Test Plan" section to your PR description. A Test Plan lets us know how these changes were tested.

Generated by 🚫 dangerJS against b69777a

@tom-un tom-un merged commit 3f1014b into microsoft:master Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tom-un tom-un tom-un approved these changes

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

@mganandraj @pull-bot @tom-un