linkedin / linkedin/eyeglass

allow forced import when import-once is enabled

Open
#99 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement package:eyeglass
Dominant language
TypeScript
Stars
738
Forks
48
PR merge metrics
No merged PRs in 30d

Description

Using ruby sass + compass and compass import once,
I sometimes had to append an exclamation mark (!) to the path to override import once.

So for having a legacy project compatible with eyeglass/libsass out of the box,
it would be helpful when eyeglass import at least tolerates
the exclamation mark at end of @imported path.

Thinking about it, may it be an useful feature to allow overriding
the import once using an exclamation mark?
Like looping over an @import or for some edge cases?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing Eyeglass's import-once handling and compare the requested trailing exclamation-mark syntax with Ruby Sass and Compass behavior. Clarify the expected result for repeated imports and edge cases, then add tests covering forced imports and normal import-once behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
sass, typescript
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.