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

Resolving Eslint Errors #634

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Commits on Mar 7, 2023

  1. 1. Changes to dev/Dockerfile

    - Added the creation of the .eslintignore file which is used to ignore the linting of javascript files.
    2. Changes to test_runner.bash
    - Indentation fixes
    FutzMonitor committed Mar 7, 2023
    Configuration menu
    Copy the full SHA
    093e9ad View commit details
    Browse the repository at this point in the history

Commits on Mar 17, 2023

  1. 1. Changes to dev/Dockerfile

    - Created an eslintignore file that ignores the Cypress tests
    FutzMonitor committed Mar 17, 2023
    Configuration menu
    Copy the full SHA
    00b6b2d View commit details
    Browse the repository at this point in the history

Commits on Mar 25, 2023

  1. 1. Changes to the .gitignore

    - Added a line to ignore the .eslintignore.
    2. Changes to the dev/Dockerfile
    - Copied the .eslint file inside the docker/dev directory into the container. Built the container locally, and this did the trick.
    - Unfortunately, I couldn't get Cypress to work on my machine to be able to verify that it works on the host machine. You only need one .eslintignore file, and it might be that the one in the docker/dev directory is good enough.
    FutzMonitor committed Mar 25, 2023
    Configuration menu
    Copy the full SHA
    7553901 View commit details
    Browse the repository at this point in the history

Commits on Mar 27, 2023

  1. Configuration menu
    Copy the full SHA
    88557fc View commit details
    Browse the repository at this point in the history
  2. 1. Changes to dev/DockerFile

    - Making sure the .eslintignore file remained hidden
    FutzMonitor committed Mar 27, 2023
    Configuration menu
    Copy the full SHA
    0d796d2 View commit details
    Browse the repository at this point in the history
  3. 1. Changes to .gitignore

    - Realized that ignoring the .eslintignore is NOT what we want. We want it to be included in the project.
    2. Addition of the dev/.eslintignore file
    - File that prevents eslint from running on js files.
    FutzMonitor committed Mar 27, 2023
    Configuration menu
    Copy the full SHA
    5adf257 View commit details
    Browse the repository at this point in the history

Commits on Mar 28, 2023

  1. 1. Changes to dev/Dockerfile

    - Moved the new COPY command below line 187 w/explaination for its existence
    FutzMonitor committed Mar 28, 2023
    Configuration menu
    Copy the full SHA
    c40fa04 View commit details
    Browse the repository at this point in the history
  2. 1. Renamed dev/.eslintignore to dev/eslintignore

    2. Changes to dev/Dockerfile
    - Changed the COPY command to make the visible eslintignore file invisible inside the dev container
    FutzMonitor committed Mar 28, 2023
    Configuration menu
    Copy the full SHA
    3fb36b0 View commit details
    Browse the repository at this point in the history

Commits on Apr 6, 2023

  1. Configuration menu
    Copy the full SHA
    c66373d View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2023

  1. Configuration menu
    Copy the full SHA
    97eef8b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    845835f View commit details
    Browse the repository at this point in the history

Commits on May 1, 2023

  1. Configuration menu
    Copy the full SHA
    0c16482 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2023

  1. Changes to component.js

    - somehow my changes were removed from my system. I had to add these files back before working. No changes here besides incorporating the stylesheet.
    Changes to BCalendarDateSelection.js
    - no noteable changes. Could not figure out how to make BCalendar work.
    Changes to BCalendarDateSelection.spec.comp.js
    - No noteable changes. Basically the same.
    FutzMonitor committed May 2, 2023
    Configuration menu
    Copy the full SHA
    9ef0d4a View commit details
    Browse the repository at this point in the history