Skip to content

feat: Adding to Sebastian Agudelo in /about #245

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Mega-Barto
Copy link

After a several mistakes (thanks @stivencardona), I added the data of Sebastian Agudelo in /about. This has been propertly tested.

@Mega-Barto
Copy link
Author

5 lines of code were never so painful xd

@stivencardona
Copy link
Contributor

@xergioalex do you now what is failing the check of pr SIZE, of @Mega-Barto ?

I think that him changes are ok!

stivencardona
stivencardona previously approved these changes Feb 27, 2025
@stivencardona
Copy link
Contributor

stivencardona commented Feb 27, 2025

I think there’s a bug in our PR calculate size step when the PR branch comes from an external repository. We’re trying to read the source branch from the origin repository, but it’s not present because the branch belongs to an external repository.

@stivencardona stivencardona added the Size - XS Between 0 and 50 lines label Feb 27, 2025
@stivencardona
Copy link
Contributor

Don't close I will fix the PR checker workflow

@stivencardona stivencardona force-pushed the adding_agudelo branch 2 times, most recently from 8ecbb26 to cf0ab2b Compare February 27, 2025 15:20
@@ -72,6 +72,11 @@ const team = [
name: 'Sergio Florez',
socialLink: 'https://www.linkedin.com/in/xergioalex/',
},
{
name: 'Sebastián Agudelo Morales',
image: 'https://i.imgur.com/2wzs36v.jpeg',
Copy link
Contributor

Choose a reason for hiding this comment

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

@Mega-Barto can you upload the image to the assets/image/people/team directory? just to avoid call an external source

Copy link
Contributor

Choose a reason for hiding this comment

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

Please resolve this, and we can merge your changes

@stivencardona stivencardona self-requested a review February 27, 2025 15:20
@stivencardona stivencardona requested a review from a team as a code owner February 27, 2025 16:10
@stivencardona stivencardona removed their request for review February 27, 2025 16:10
@stivencardona stivencardona self-requested a review February 27, 2025 16:16
@stivencardona stivencardona dismissed their stale review February 27, 2025 16:16

Error approve

@stivencardona stivencardona removed their request for review February 27, 2025 16:16
@stivencardona
Copy link
Contributor

@Mega-Barto Please fix the pending change or the PR will be declined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size - XS Between 0 and 50 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants