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 Custom Label Addition package.xml generation #114

Merged
merged 1 commit into from
Apr 2, 2021

Conversation

scolladon
Copy link
Owner

What does this pull request contains? Explain your changes.


  • 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.

Explain your changes


Deal with added files the same as modified ones.
Previously the code red the information from the file in the file system for added file.
Which is inaccurate when the --to parameter is different of the HEAD.
This PR define the content of package.xml (and delta generation content) from the diff (not from the content of the current file)

Does this close any currently open issues?


closes #47

  • Jest test to check the fix is applied are added.

Where has this been tested?


Operating System: Darwin Kernel Version 18.7.0: Tue Jan 12 22:04:47 PST 2021; root:xnu-4903.278.56~1/RELEASE_X86_64

yarn version: 1.22.10

node version: v15.12.0

git version: 2.31.0

sfdx version: sfdx-cli/7.93.1 darwin-x64 node-v15.12.0

sgd plugin version: sfdx-git-delta 4.2.3

Remove not used anymore _fillPackageFromFile
Change the API _handleInFile => _handleInDiff (more accurate)
@scolladon scolladon merged commit 1447b5c into master Apr 2, 2021
@scolladon scolladon deleted the fix/custom-label-addition branch April 2, 2021 12:35
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.

[Labels Directory] CustomLabel file doesn't contains label changes
2 participants