Skip to content

Conversation

@Sowmiya07
Copy link
Contributor

Description

Add certification automation

Change Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Mentor Update
  • Data Update
  • Documentation
  • Other

Related Issue

Screenshots

image

Pull request checklist

Please check if your PR fulfills the following requirements:

  • I checked and followed the contributor guide
  • I have tested my changes locally.
  • I have added a screenshot from the website after I tested it locally

@Sowmiya07 Sowmiya07 requested a review from a team as a code owner December 20, 2025 16:34
Copy link
Contributor

@khairahscorner khairahscorner left a comment

Choose a reason for hiding this comment

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

Let's add some tests for confidence; also, can we move the folder as a sub-folder under "tools"? These are where other automation scripts we use are placed.
Otherwise LGTM🙌

Copy link
Contributor

Choose a reason for hiding this comment

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

Can we write some tests too?

Copy link
Contributor

Choose a reason for hiding this comment

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

+1

Very important!

@@ -0,0 +1,3 @@
TestFN TestLN
Copy link
Contributor

Choose a reason for hiding this comment

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

please move this folder under tools and update README there to include this extra automation. Thanks for the amazing script, I will give a try locally also, to extend and create another type of certificate.

{
"certificate_types": [
{
"type": "mentee",
Copy link
Contributor

Choose a reason for hiding this comment

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

great idea! Looking forward to you merge it and I can extend to new certificates :)

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