You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The mintpy/sh/load_data_aoi.sh script is supposed to work with a standard unix sh shell but it actually contain some statements using a bash specific syntax.
$ sh -n mintpy/sh/load_data_aoi.sh
mintpy/sh/load_data_aoi.sh: 49: Syntax error: redirection unexpected
Reminders
Fix #xxxx
Pass Codacy code review (green)
Pass Circle CI test (green)
Make sure that your code follows our style. Use the other functions/files as a basis.
If modifying functionality, describe changes to function behavior and arguments in a comment below the function declaration.
If adding new functionality, add a detailed description to the documentation and/or an example.
💖 Thanks for opening this pull request! Please check out our contributing guidelines. 💖
Keep in mind that all new features should be documented. It helps to write the comments next to the code or below your functions describing all arguments, and return types before writing the code. This will help you think about your code design and usually results in better code.
Just for my curiosity, what is the purpose of having the mintpy/sh/*.sh scripts inside the mintpy Python package folder?
It seems to me that none of them is used in the Python code, so probably it could be a good idea to move them to another place.
Do I miss something?
You are right. Only sh/plot_smallbaselineApp.sh was used before in the code, but not anymore. I like the idea of moving them to another place. Do you have any suggestions for it?
You are right. Only 'sh/plot_smallbaselineApp.sh` was used before in the code, but not anymore. I like the idea of moving them to another place. Do you have any suggestions for it?
Probably it could be a scripts folder in the project root.
If you need to install those scripts, you could use the scripts argument of setuptools.setup.
In that case, anyway, I recommend to prefix your scripts with e.g. mintpy- in order to make it easy to recognize mintpy related stuff in the global /usr/bin/ folder (on unix at least).
If you want I can prepare a patch for it, but it will be not before a week.
I will be traveling the next week.
Having them in a scripts folder sounds good to me. Since they are not used currently, we don't need to install them. One could easily find them in the repo if they want to use them independently. I will wait for your PR then.
yunjunz
changed the title
Use the correct shell in the shebang string
use sh instead of bash in load_data_aoi.shMay 22, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use the correct shell in the shebang string
The
mintpy/sh/load_data_aoi.shscript is supposed to work with a standard unixshshell but it actually contain some statements using abashspecific syntax.Reminders