Skip to content

Fix replacing every src string / Issue from Weblinks repo #376

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mahmoudmagdy1-1
Copy link

Pull Request for Issue 591 from weblinks.

Summary of Changes

When using patchtester to test components, it replaces every /src folder with empty string (it removes it from the path) but in reality we only need to remove the first /src occurrence only, otherwise if the the pull request we checking the changes from has another /src in it's path its going to get removed and we get this error
The file marked for modification does not exist, like the issue in the link above

Testing Instructions

  1. Login as Admin
  2. Fetch data from the Weblinks GitHub Repository using patchtester
  3. Apply a patch from any pull request that includes file path changes containing /src

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.

1 participant