Merge pull request #325 from devicons/TB_bugFix

Fixed an async issue in the peek workflow and added a comment action

Author: Thomas-Boi

.github/scripts/build_assets/SeleniumRunner.py

@@ -75,6 +75,14 @@ def set_options(self, download_path: str, geckodriver_path: str,
         self.driver = WebDriver(options=options, executable_path=geckodriver_path)
         self.driver.get(self.ICOMOON_URL)
         assert "IcoMoon App" in self.driver.title
+        
+        # wait until the whole web page is loaded by testing the hamburger input
+        hamburger_input = WebDriverWait(self.driver, SeleniumRunner.LONG_WAIT_IN_SEC).until(
+            ec.element_to_be_clickable((By.CSS_SELECTOR,
+                "button.btn5.lh-def.transparent i.icon-menu"))
+        )
+        hamburger_input.click()
+        print("Accessed icomoon.io")
 
     def upload_icomoon(self, icomoon_json_path: str):
         """
@@ -86,7 +94,7 @@ def upload_icomoon(self, icomoon_json_path: str):
         try:
             # find the file input and enter the file path
             import_btn = WebDriverWait(self.driver, SeleniumRunner.LONG_WAIT_IN_SEC).until(
-                ec.presence_of_element_located((By.CSS_SELECTOR, "div#file input"))
+                ec.element_to_be_clickable((By.CSS_SELECTOR, "div#file input"))
             )
             import_btn.send_keys(icomoon_json_path)
         except Exception as e:
@@ -138,6 +146,7 @@ def upload_svgs(self, svgs: List[str]):
                 ec.element_to_be_clickable((By.XPATH, "//button[text()='Select All']"))
             )
             select_all_button.click()
+            print("Finished uploading the svgs...")
         except Exception as e:
             self.close()
             raise e
@@ -155,7 +164,7 @@ def click_hamburger_input(self):
             )
 
             menu_appear_callback = ec.element_to_be_clickable(
-                (By.CSS_SELECTOR, "h1#setH2 ul")
+                (By.CSS_SELECTOR, "h1 ul.menuList2")
             )
 
             while not menu_appear_callback(self.driver):

.github/workflows/peek_icons.yml

@@ -37,4 +37,17 @@ jobs:
         if: ${{success()}}
         with:
           name: new_icons
-          path: ./new_icons.png
+          path: ./new_icons.png
+      - name: Comment on the PR about the result
+        uses: github-actions-up-and-running/pr-comment@v1.0.1
+        with:
+          repo-token: ${{ secrets.GITHUB_TOKEN }}
+          message: >
+            Hi! I'm Devicons' GitHub Actions Bot!
+            
+            I just peeked at the icons that you wanted to add and upload them to the 
+            [Actions page](https://github.com/devicons/devicon/actions). The maintainers
+            will now take a look at it and decide whether to merge your PR.
+
+            Cheers :),
+            Bot

devicon.json

@@ -1469,8 +1469,7 @@
                 "plain"
             ],
             "font": [
-                "line",
-                "plain"
+                "line"
             ]
         },
         "color": "#F0D6B7",