search

Kyonru / Assets-Renamer

Change the name of material design assets to be used on a react native project
1 stars 0 forks source link

Sanitize path #6

Closed davidtavarez closed 6 years ago

davidtavarez commented 6 years ago

The script should verify and sanitize the destination path properly.

hectoraristy commented 6 years ago

I think we can validate this by looking if the last character of the path does has a '/'. What do you think @davidtavarez @Kyonru ?

davidtavarez commented 6 years ago

We could add os.path.abspath(args.path + '/' + directory) here https://github.com/Kyonru/Assets-Renamer/blob/d58cf6b50376b09de3f2a8b57ad18a8d084324ac/RenamerMaterialDesign.py#L59

davidtavarez commented 6 years ago

@Kyonru I can do it.

davidtavarez commented 6 years ago

PR placed https://github.com/Kyonru/Assets-Renamer/pull/14