Skip to content

Commit 2b228d5

Browse files
sanjay-kvsanjay-kv
authored
Merge pull request #961 from sohumvenkatadri7/fix-interviewprep-featurestext
Fix badge colors in light/dark mode.
2 parents 6066bb0 + 9487b0c commit 2b228d5

File tree

2 files changed

+118
-58
lines changed

2 files changed

+118
-58
lines changed

‎.github/workflows/pr-issue-sync.yml‎

Lines changed: 60 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
name: Sync PR data from Linked Issues
1+
name: PR Issue Sync
22

33
on:
4-
pull_request_target:
5-
types: [opened, edited, synchronize]
4+
workflow_dispatch: {} # manual trigger only (disables automatic runs)
65

76
jobs:
87
sync-metadata:
@@ -28,66 +27,69 @@ jobs:
2827
}
2928
core.setOutput("issues", JSON.stringify(matches));
3029
31-
- name: Post or Update data Comment
32-
if: steps.extract.outputs.issues && steps.extract.outputs.issues != '[]'
30+
- name: Sync metadata
3331
uses: actions/github-script@v7
3432
with:
3533
github-token: ${{ secrets.GITHUB_TOKEN }}
3634
script: |
37-
const issues = JSON.parse(`${{ steps.extract.outputs.issues }}`);
38-
const prNumber = context.payload.pull_request.number;
39-
40-
let combinedLabels = [];
41-
let combinedAssignees = [];
42-
let combinedMilestones = [];
43-
44-
for (const number of issues) {
35+
const data = ${{ fromJSON(format('{{\"issues\":{0},\"pr\":{1}}}', steps.extract.outputs.issues, github.event.pull_request.number)) }};
36+
const prNumber = data.pr;
37+
const issueNumbers = data.issues || [];
38+
39+
if (issueNumbers.length === 0) {
40+
console.log("No linked issues found");
41+
return;
42+
}
43+
44+
for (const issueNumber of issueNumbers) {
4545
try {
46-
const issue = await github.rest.issues.get({
47-
...context.repo,
48-
issue_number: number
46+
// Fetch issue
47+
const { data: issue } = await github.rest.issues.get({
48+
owner: context.repo.owner,
49+
repo: context.repo.repo,
50+
issue_number: parseInt(issueNumber)
4951
});
50-
51-
combinedLabels.push(...issue.data.labels.map(l => l.name));
52-
combinedAssignees.push(...issue.data.assignees.map(a => a.login));
53-
if (issue.data.milestone) combinedMilestones.push(issue.data.milestone.title);
54-
55-
} catch (err) {
56-
console.log(`Could not fetch issue #${number}: ${err.message}`);
52+
53+
console.log(`Syncing metadata from Issue #${issueNumber} to PR #${prNumber}`);
54+
55+
// --- Sync Labels ---
56+
const issueLabels = issue.labels.map(l => l.name);
57+
const { data: pr } = await github.rest.pulls.get({
58+
owner: context.repo.owner,
59+
repo: context.repo.repo,
60+
pull_number: prNumber
61+
});
62+
const currentPRLabels = pr.labels.map(l => l.name);
63+
const combinedLabels = Array.from(new Set([...currentPRLabels, ...issueLabels]));
64+
65+
await github.rest.issues.addLabels({
66+
owner: context.repo.owner,
67+
repo: context.repo.repo,
68+
issue_number: prNumber,
69+
labels: combinedLabels
70+
});
71+
console.log(`Labels applied: ${combinedLabels.join(', ')}`);
72+
73+
// --- Sync Milestone ---
74+
if (issue.milestone) {
75+
await github.rest.issues.update({
76+
owner: context.repo.owner,
77+
repo: context.repo.repo,
78+
issue_number: prNumber,
79+
milestone: issue.milestone.number
80+
});
81+
console.log(`Milestone synced: ${issue.milestone.title}`);
82+
}
83+
84+
// --- Optionally: Add a comment on PR ---
85+
await github.rest.issues.createComment({
86+
owner: context.repo.owner,
87+
repo: context.repo.repo,
88+
issue_number: prNumber,
89+
body: `✅ Synchronized metadata from Issue #${issueNumber}:\nLabels: ${issueLabels.join(', ')}\nMilestone: ${issue.milestone ? issue.milestone.title : 'None'}`
90+
});
91+
92+
} catch (error) {
93+
console.error(`Error syncing issue #${issueNumber} to PR #${prNumber}:`, error.message);
5794
}
5895
}
59-
60-
// Deduplicate
61-
combinedLabels = [...new Set(combinedLabels)];
62-
combinedAssignees = [...new Set(combinedAssignees)];
63-
combinedMilestones = [...new Set(combinedMilestones)];
64-
65-
const commentBody =
66-
`### Synced data from Linked Issues\n\n` +
67-
`**Labels:**\n${combinedLabels.length ? combinedLabels.map(l => `- ${l}`).join("\n") : "- None"}\n\n` +
68-
`**Assignees:**\n${combinedAssignees.length ? combinedAssignees.map(a => `- ${a}`).join("\n") : "- None"}\n\n` +
69-
`**Milestones:**\n${combinedMilestones.length ? combinedMilestones.map(m => `- ${m}`).join("\n") : "- None"}\n`;
70-
71-
// Get existing comments
72-
const comments = await github.rest.issues.listComments({
73-
...context.repo,
74-
issue_number: prNumber
75-
});
76-
77-
// Find existing workflow comment
78-
const existingComment = comments.data.find(c => c.body.includes("### Synced data from Linked Issues"));
79-
80-
if (existingComment) {
81-
await github.rest.issues.updateComment({
82-
...context.repo,
83-
comment_id: existingComment.id,
84-
body: commentBody
85-
});
86-
} else {
87-
// Create new comment
88-
await github.rest.issues.createComment({
89-
...context.repo,
90-
issue_number: prNumber,
91-
body: commentBody
92-
});
93-
}

‎src/css/custom.css‎

Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1517,3 +1517,61 @@ a {
15171517
.theme-doc-breadcrumbs a.breadcrumbs__link > svg.breadcrumbHomeIcon_YNFT {
15181518
display: inline-block;
15191519
}
1520+
1521+
/* Fix difficulty and topic pill backgrounds in light/dark mode */
1522+
[data-theme='light'] .rounded.bg-gray-100,
1523+
[data-theme='light'] .rounded-full.bg-green-100,
1524+
[data-theme='light'] .rounded-full.bg-yellow-100,
1525+
[data-theme='light'] .rounded-full.bg-red-100,
1526+
[data-theme='light'] .rounded-full.bg-blue-100 {
1527+
background-color: var(--ifm-color-emphasis-100) !important;
1528+
}
1529+
1530+
[data-theme='light'] .rounded-full.bg-green-100 {
1531+
background-color: #dcfce7 !important; /* green-100 */
1532+
}
1533+
1534+
[data-theme='light'] .rounded-full.bg-yellow-100 {
1535+
background-color: #fef9c3 !important; /* yellow-100 */
1536+
}
1537+
1538+
[data-theme='light'] .rounded-full.bg-red-100 {
1539+
background-color: #fee2e2 !important; /* red-100 */
1540+
}
1541+
1542+
[data-theme='light'] .rounded-full.bg-blue-100 {
1543+
background-color: #dbeafe !important; /* blue-100 */
1544+
}
1545+
1546+
[data-theme='light'] .rounded.bg-gray-100 {
1547+
background-color: #f3f4f6 !important; /* gray-100 */
1548+
}
1549+
1550+
/* Dark mode overrides */
1551+
[data-theme='dark'] .rounded.dark\:bg-gray-700,
1552+
[data-theme='dark'] .rounded-full.dark\:bg-green-900,
1553+
[data-theme='dark'] .rounded-full.dark\:bg-yellow-900,
1554+
[data-theme='dark'] .rounded-full.dark\:bg-red-900,
1555+
[data-theme='dark'] .rounded-full.dark\:bg-blue-900 {
1556+
opacity: 1 !important;
1557+
}
1558+
1559+
[data-theme='dark'] .rounded-full.dark\:bg-green-900 {
1560+
background-color: #14532d !important; /* green-900 */
1561+
}
1562+
1563+
[data-theme='dark'] .rounded-full.dark\:bg-yellow-900 {
1564+
background-color: #713f12 !important; /* yellow-900 */
1565+
}
1566+
1567+
[data-theme='dark'] .rounded-full.dark\:bg-red-900 {
1568+
background-color: #7f1d1d !important; /* red-900 */
1569+
}
1570+
1571+
[data-theme='dark'] .rounded-full.dark\:bg-blue-900 {
1572+
background-color: #1e3a8a !important; /* blue-900 */
1573+
}
1574+
1575+
[data-theme='dark'] .rounded.dark\:bg-gray-700 {
1576+
background-color: #374151 !important; /* gray-700 */
1577+
}

0 commit comments

Comments
 (0)