Skip to content

get gt from airtable#399

Merged
drmowinckels merged 16 commits intomainfrom
gtget
Apr 17, 2025
Merged

get gt from airtable#399
drmowinckels merged 16 commits intomainfrom
gtget

Conversation

@drmowinckels
Copy link
Member

No description provided.

Copy link
Member Author

@drmowinckels drmowinckels left a comment

Choose a reason for hiding this comment

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

Intention is to periodically retrieve updated information on the global team from Airtable.

Alumni are stored on website and deleted from airtable.

@github-actions
Copy link
Contributor

🎉 The preview is built! Check it out 👀

@github-actions
Copy link
Contributor

Test build failed 😭 Check out the build logs.

@github-actions
Copy link
Contributor

🎉 The preview is built! Check it out 👀

@drmowinckels drmowinckels marked this pull request as ready for review March 24, 2025 11:09
@drmowinckels drmowinckels requested review from a team as code owners March 24, 2025 11:09
@drmowinckels
Copy link
Member Author

drmowinckels commented Mar 24, 2025

both @cosimameyer and @SoyAndrea please take a look. Its important we all understand what the changes mean and what I've done :)

  • The folder data/global_team/ is now updated with the script scripts/get_global_team.R. Which is a script grabbing data from Airtable. This way, we can have a singular source of global team members and their tasks to update at any given time.
  • the workflow in GitHub Actions get_global_team.yml runs every sunday, or when triggered manually.
  • Global Team members that retire are grabbed from Airtable, and stored in the data/global_team/alumni folder, and then deleted from AIrtable, for housekeeping.

In this PR I have also made sure we could update Hugo to a newer version.
Larger changes is:

  • using the new try functionality when attempting to get images for the r-ladies blogs
  • fixing hugo code for the directory to never include empty lists, as this now would break the page.

@github-actions
Copy link
Contributor

🎉 The preview is built! Check it out 👀

@github-actions
Copy link
Contributor

🎉 The preview is built! Check it out 👀

@github-actions
Copy link
Contributor

🎉 The preview is built! Check it out 👀

@drmowinckels drmowinckels requested a review from Copilot April 17, 2025 19:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements an automated process to retrieve global team data from Airtable and update the website configuration accordingly. Key changes include adjustments to the Hugo configuration files, new GitHub Actions workflows for installing dependencies and running an R script to retrieve data, and modifications to the R setup steps in multiple workflows.

Reviewed Changes

Copilot reviewed 112 out of 115 changed files in this pull request and generated no comments.

Show a summary per file
File Description
config/_default/markup.yaml Updated Goldmark configuration settings.
config/_default/hugo.yaml Added ignoreLogs for Goldmark warnings and reintroduced outputs config.
.github/workflows/global-team.yml New workflow steps to install dependencies, run the R script, and push changes.
.github/workflows/check-jsons.yaml Updated the R version parameter to use "renv".
.github/workflows/chapters-merge.yaml Updated the R version parameter to use "renv".
.github/workflows/build-production.yaml Updated the R version parameter to use "renv" and a naming typo in a step.
Files not reviewed (3)
  • .hugoversion: Language not supported
  • DESCRIPTION: Language not supported
  • content/.gitignore: Language not supported
Comments suppressed due to low confidence (1)

.github/workflows/build-production.yaml:46

  • The step name 'Serup renv' appears to be a misspelling; consider renaming it to 'Setup renv'.
- name: Serup renv

@drmowinckels drmowinckels merged commit 8daa282 into main Apr 17, 2025
0 of 2 checks passed
@drmowinckels drmowinckels deleted the gtget branch April 17, 2025 22:03
@drmowinckels drmowinckels mentioned this pull request Jun 1, 2025
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.

2 participants