Skip to content

Commit

Permalink
feat(ui): Always enable Analyzer job for an ORT Run
Browse files Browse the repository at this point in the history
Currently, the Analyzer job needs to always be included in an ORT run,
so make it mandatory in the run creation form.

As there is a pending issue about uploading pre-created Analyzer results
for a run [1], we implement this with minimal changes to the UI, by
always enabling the Analyzer job switch and preventing to disable it.
This prepares for future changes to the run creation form regarding
the upload functionality.

[1]: eclipse-apoapsis#193

Signed-off-by: Jyrki Keisala <[email protected]>
  • Loading branch information
Etsija committed May 28, 2024
1 parent 0624e5b commit 7881da6
Showing 1 changed file with 8 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,13 @@ const CreateRunPage = () => {
/>

<h3 className='mt-4'>Enable and configure jobs</h3>
<div className='text-sm text-gray-500'>
Configure the jobs to be included in the ORT Run.{' '}
</div>
<div className='text-sm text-gray-500'>
NOTE: Currently, the Analyzer needs to always run as part of an
ORT Run.
</div>

<Accordion type='multiple'>
{/* Analyzer job */}
Expand All @@ -287,6 +294,7 @@ const CreateRunPage = () => {
<Switch
className='my-4 mr-4 data-[state=checked]:bg-green-500'
checked={field.value}
disabled
onCheckedChange={field.onChange}
/>
</FormControl>
Expand Down

0 comments on commit 7881da6

Please sign in to comment.