Skip to content
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

Use array for positions #112

Closed
wants to merge 2 commits into from
Closed

Conversation

jahzielv
Copy link
Contributor

What this PR does / why we need it:
Changes position to be an array of strings representing different job positions.

fixes #86

@jahzielv
Copy link
Contributor Author

This PR currently only changes work.position from a string to an array. position is a member of several other sections (references, volunteer); do we want to change all of those as well?

@jahzielv
Copy link
Contributor Author

Any ideas as to why this build is failing? According to Travis:

--- FAIL: TestRenderRun (0.21s)
    render_test.go:23: Could not render the resume html: remote repository is empty

Why would the render command create this error? I've been tracing through calls but haven't found anything that sticks out.

@stp-ip
Copy link
Member

stp-ip commented Oct 19, 2019

Did you look into: https://github.com/resumic/schema/blob/a810c874c164e3439e47211f925e4e47141cf1eb/cmd/render_test.go?

@stp-ip
Copy link
Member

stp-ip commented Oct 19, 2019

Happy to take a deeper dive in the coming days, if necessary. Also looping in @ArmanMazdaee and @erbesharat for additional help, if needed.

@stp-ip
Copy link
Member

stp-ip commented Nov 12, 2019

Just checking in. Did the render_test.go file help in figuring out the failing tests?

@ArmanMazdaee Care to take a look?

@jahzielv jahzielv closed this Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Use array for positions?
2 participants