Allow dots in environment names (specifically, in version numbers).

[mjkw31]

Modified the name validation regexp to allow dots after the first character (to prevent hidden files, and parent directory refs).

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (4a14936) 63.13% compared to head (e7583c2) 63.13%.

Files	Patch %	Lines
softpack_core/schemas/environment.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           develop      #47   +/-   ##
========================================
  Coverage    63.13%   63.13%           
========================================
  Files           17       17           
  Lines          868      868           
  Branches       153      153           
========================================
  Hits           548      548           
  Misses         313      313           
  Partials         7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sersorrel]

This will cause a problem with #48 (https://github.com/wtsi-hgi/softpack-core/pull/48/files#diff-7d1716a6cc7338126394855a08f774de9bf287534431df547fc03c25fce75317R375-R378):

            m = re.fullmatch(r"^(.*)-(\d+)$", env.name)
            if not m:
                raise Exception
            versionless_name, version = m.groups()

This should be fine – the integer version number is unconditionally appended for environments created via the GraphQL createEnvironment endpoint, so this just allows dots in names, and as a side-effect permits version numbers with dots to be passed to createFromModule.

[sersorrel]

It's unfortunate that the new test doesn't get run in CI (because the whole of test_environment.py has the repo mark) – frustratingly it doesn't seem like there's a way to "unmark" a specific test when the whole file is marked.