Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style: fix lint issues #740

Merged
merged 2 commits into from
Jun 22, 2020
Merged

style: fix lint issues #740

merged 2 commits into from
Jun 22, 2020

Conversation

hkang1
Copy link
Contributor

@hkang1 hkang1 commented Jun 18, 2020

Description

Test Plan

Commentary (optional)

export type RecentTask = AnyTask & RecentEvent;
export type RecentCommandTask = CommandTask & RecentEvent
export type RecentExperimentTask = ExperimentTask & RecentEvent
export type RecentCommandTask = CommandTask & RecentEvent;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we make these Error level failures so they break CI test?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • the multiple space rule.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep I like that. I wasn't sure if that was too severe but with auto fix I think that's reasonable.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll update this PR to make it error level.

Copy link
Member

@hamidzr hamidzr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thoughts on making them error level?

@hamidzr hamidzr assigned hkang1 and unassigned hamidzr Jun 18, 2020
@hkang1 hkang1 assigned hamidzr and unassigned hkang1 Jun 18, 2020
@hamidzr hamidzr assigned hkang1 and unassigned hamidzr Jun 18, 2020
@hkang1 hkang1 merged commit d9f443b into determined-ai:master Jun 22, 2020
@hkang1 hkang1 deleted the lint-fixes branch June 22, 2020 17:57
tayritenour pushed a commit to tayritenour/determined that referenced this pull request Apr 25, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit to eecsliu/determined that referenced this pull request Jun 23, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
stoksc pushed a commit that referenced this pull request Jun 26, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit that referenced this pull request Jun 28, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit that referenced this pull request Jun 28, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
stoksc pushed a commit that referenced this pull request Jul 20, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit that referenced this pull request Jul 24, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
stoksc pushed a commit that referenced this pull request Oct 17, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
azhou-determined pushed a commit that referenced this pull request Dec 7, 2023
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
wes-turner pushed a commit that referenced this pull request Feb 2, 2024
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
@dannysauer dannysauer added this to the 0.12.10 milestone Feb 6, 2024
rb-determined-ai pushed a commit that referenced this pull request Feb 29, 2024
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
amandavialva01 pushed a commit that referenced this pull request Mar 18, 2024
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit that referenced this pull request Apr 18, 2024
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
eecsliu pushed a commit to determined-ai/determined-release-testing that referenced this pull request Apr 22, 2024
Remove podman integration tests from gate runs as they are not reliable.
Disable by breaking inheritance of steps -- use test-e2e-slurm-disabled workflow that does nothing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants