Skip to content

fix: resolve obsidian community guidelines and lint errors - #7

Merged
ClaudiaFang merged 11 commits into
masterfrom
fix/obsidian-compliance
Apr 25, 2026
Merged

ClaudiaFang merged 11 commits into
masterfrom
fix/obsidian-compliance

Conversation

@ClaudiaFang

Copy link
Copy Markdown
Member

This PR fixes several issues identified by the Obsidian automated scan:

  • Replaced native confirm() with custom ConfirmModal
  • Updated UI strings to use Sentence case
  • Fixed lint errors by adopting official obsidian-sample-plugin configuration
  • Removed all eslint-disable comments and fixed underlying code issues
  • Fixed a TypeScript compilation error in SyncStatusView.ts
  • Resolved parsing errors in vitest.config.ts via tsconfig.json update

Comment thread .github/workflows/sonarqube.yml Fixed
Comment thread .github/workflows/check.yml Fixed
Comment thread .github/workflows/reusable-check.yml Fixed
@ClaudiaFang ClaudiaFang added the wontfix This will not be worked on label Apr 25, 2026
@ClaudiaFang
ClaudiaFang enabled auto-merge April 25, 2026 04:17
Comment thread .github/workflows/sonarqube.yml Fixed
@ClaudiaFang
ClaudiaFang disabled auto-merge April 25, 2026 04:22
@ClaudiaFang
ClaudiaFang enabled auto-merge (squash) April 25, 2026 04:22
@ClaudiaFang
ClaudiaFang merged commit 7bef2ca into master Apr 25, 2026
9 of 10 checks passed
@ClaudiaFang
ClaudiaFang deleted the fix/obsidian-compliance branch April 25, 2026 04:22
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
3.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Comment on lines +12 to +31
name: Build
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: '22'
cache: 'npm'
- run: npm ci
- run: npm run test -- --coverage
- run: npm run build

# Upload coverage for the next job
- name: Upload coverage
uses: actions/upload-artifact@v4
with:
name: coverage-report
path: coverage/

sonar:
Comment on lines +32 to +47
name: SonarQube
needs: build
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Download coverage
uses: actions/download-artifact@v4
with:
name: coverage-report
path: coverage/
- name: SonarQube Scan
uses: SonarSource/sonarqube-scan-action@v7.1.0
env:
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ClaudiaFang

Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 1.1.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ClaudiaFang

Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants