Pre-commit blocks merge on broken links

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Build
    • None
    • None
    • Go Drivers
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Context

      Dead links should not fail pre-commit since we have no power over when a site goes down or for how long. For example, https://goreportcard.com/ is down and we don't know why. We don't want to replace these links unless we must. However, it being down is blocking PR merging.

      Definition of done

      Convert the dead links failure to a warning to allow resolution for such problems to go through the normal development cycle.

      Separate this check into a standalone CI task that is not required for merge. 

       

      Create a ticket to migrate off https://goreportcard.com/ if it being down is permanent. (See GODRIVER-4007)

      Pitfalls

      NA

        There are no Sub-Tasks for this issue.

            Assignee:
            Unassigned
            Reporter:
            Preston Vasquez
            None
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: