lint views/schedulers, views/metrics, views/lti_nrps, views/lti_config, views/extensions

[20wildmanj]

Summary

Summary generated by Reviewpad on 18 Sep 23 15:30 UTC

This pull request includes the following changes:

• In the file app/views/lti_config/index.html.erb, the changes handle the case when @lti_config_hash is nil and improve code readability and consistency.
• In the file app/views/metrics/index.html.erb, there are multiple changes including modifying variable values, adding autocomplete attributes, and formatting changes.
• In the file sync_roster.html.erb, a space has been added before the closing curly brace in the locals hash.
• In the file index.html.erb, changes include updating a method, modifying indentation, and improving code readability.
• In the file edit.html.erb, the changes include indentation, adding a builder option, and adding help text and placeholders for form fields.
• In the file index.html.erb in the app/views/extensions directory, there are changes to the script tag, table rows, table cells, and link to delete an extension.
• In the file new.html.erb in the app/views/schedulers directory, changes include updating help text attributes, adding default values, and including a checkbox attribute.

These changes aim to improve code readability, handle edge cases, and provide more informative help text.

Description

• lint views/schedulers, views/metrics, views/lti_nrps, views/lti_config, views/extensions using erblint using bundle exec erblint app/views/*(folder)/*(file).html.erb -a

Motivation and Context

• Part of summer linting

How Has This Been Tested?

• go to each individual page, ensure that it renders properly

Checklist:

• I have run rubocop for style check. If you haven't, run overcommit --install && overcommit --sign to use pre-commit hook for linting

[michellexliu]

lgtm!