Skip to content

Conversation

@jhoward-lm
Copy link
Contributor

@jhoward-lm jhoward-lm commented Apr 30, 2024

Fix validation for input URLs

Description

  • Fix incorrect URL matching when selecting the fetcher to use
  • Remove on:push rules for workflows triggering duplicate runs
  • Enable the paralleltest and testpackage linters, update current test files to accommodate

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Signed-off-by: Jonathan Howard <jonathan.w.howard@lmco.com>
@jhoward-lm jhoward-lm added the bug Something isn't working label Apr 30, 2024
@jhoward-lm jhoward-lm self-assigned this Apr 30, 2024
Signed-off-by: Jonathan Howard <jonathan.w.howard@lmco.com>
@jhoward-lm jhoward-lm merged commit 10e558a into main Apr 30, 2024
@jhoward-lm jhoward-lm deleted the url-parse-fix branch April 30, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants