Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

Fix issue with docs lint #1102

Merged
merged 2 commits into from
Oct 11, 2022
Merged

Fix issue with docs lint #1102

merged 2 commits into from
Oct 11, 2022

Conversation

meain
Copy link
Member

@meain meain commented Oct 10, 2022

Description

Had small issue in #1052 regarding linting docs. This fixes that. Also fixes a log artifact publish for failed tests.

Type of change

  • 🌻 Feature
  • πŸ› Bugfix
  • πŸ—ΊοΈ Documentation
  • πŸ€– Test
  • πŸ’» CI/Deployment
  • 🐹 Trivial/Minor

Issue(s)

Test Plan

  • πŸ’ͺ Manual
  • ⚑ Unit test
  • πŸ’š E2E

@meain meain temporarily deployed to Testing October 10, 2022 05:43 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 05:44 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 05:44 Inactive
@meain meain requested review from vkamra and ryanfkeepers October 10, 2022 05:44
@meain meain temporarily deployed to Testing October 10, 2022 05:45 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:00 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:00 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:00 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:00 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:24 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:24 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:24 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:24 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:31 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:32 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 06:32 Inactive
@meain meain mentioned this pull request Oct 10, 2022
9 tasks
@meain meain temporarily deployed to Testing October 10, 2022 06:33 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 07:04 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 07:04 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 07:04 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 07:05 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 08:21 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 08:21 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 08:21 Inactive
@meain meain temporarily deployed to Testing October 10, 2022 08:21 Inactive
@meain
Copy link
Member Author

meain commented Oct 10, 2022

Test-Sute is failing. I'm not sure why this is happening but it seems to happening in a few other commits in main even before the CI rework PR got merged. Let me know if someone has any idea what might be going on.

@meain meain temporarily deployed to Testing October 11, 2022 02:59 Inactive
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!Β  Β  Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@meain meain temporarily deployed to Testing October 11, 2022 03:00 Inactive
@meain meain temporarily deployed to Testing October 11, 2022 03:00 Inactive
@meain meain temporarily deployed to Testing October 11, 2022 03:00 Inactive
@meain meain temporarily deployed to Testing October 11, 2022 03:01 Inactive
@meain meain changed the title Fix an issue with docs lint Fix issue with docs lint Oct 11, 2022
@meain meain merged commit 62c704a into main Oct 11, 2022
@meain meain deleted the fix-docs-lint branch October 11, 2022 03:20
ryanfkeepers pushed a commit that referenced this pull request Oct 11, 2022
* Fix docs lint

* Fix log upload path
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants