Skip to content

Conversation

@NotTacoz
Copy link
Contributor

@NotTacoz NotTacoz commented Jul 7, 2025

alice was repeated twice

@NotTacoz NotTacoz requested review from Copilot and jjustisk July 7, 2025 15:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR corrects a duplicated committee member entry and enhances the build configuration with explicit SASS loader options.

  • Replaces an incorrectly repeated “Austin Delic” with the correct profile
  • Removes the duplicate <CloudProfile> block for Austin
  • Adds loaders.scss settings in nuxt.config.js to specify the Sass implementation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pages/about.vue Updated committee member list: fixed name and removed duplicate
nuxt.config.js Added SASS loader configuration (implementation and options)
Comments suppressed due to low confidence (2)

nuxt.config.js:13

  • [nitpick] Add a comment explaining why fiber is set to false under sassOptions to clarify the motivation behind disabling fiber.
                    fiber: false

pages/about.vue:99

  • [nitpick] Consider refactoring the hardcoded list of committee members into an array and using v-for to render <CloudProfile> components. This would reduce repetition and make updates easier.
    <b-row align-h="around" class="mt-3 text-center">

Copy link
Contributor

@jjustisk jjustisk left a comment

Choose a reason for hiding this comment

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

no edit on .js file

@jjustisk jjustisk merged commit 3c598f8 into main Jul 8, 2025
2 checks passed
@jjustisk jjustisk deleted the update/committee-page branch July 8, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants