replace . with _ for contract names

[tippenein]

Description

fixes #1031 by replacing . with _ in contract file names

Breaking change?

N/A

Example

---

Checklist

• Tests added in this PR (if applicable)

If there are applicable tests to add, I'm glad to do that. I was unable to find them tho, so I added the screenshots above

[CLAassistant]

All committers have signed the CLA.

[tippenein]

Still looking for documentation on what characters are allowed for clarity file names, but will update if I find anything beyond what was reported in the issue

[tippenein]

Still looking for documentation on what characters are allowed for clarity file names, but will update if I find anything beyond what was reported in the issue

It looks like you can get around the check if you use " but this seems outside of plausible use.

cargo run contracts new char."?"

will create a new contract called char_? but won't be able to run. Seems like anyone doing this would expect trouble though..

[hugocaillard]

I think it makes sense as it is.

For other unexpected use cases, we probably want to throw an error instead. It can be addressed in a future PR.

Can you sign CLA please?

[hugocaillard]

LGTM, thanks a lot for your contribution @tippenein