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

[UserStory] Add workarounds to Contributing.md #201

Open
5 of 8 tasks
Aleksander-Gomez opened this issue Oct 4, 2023 · 8 comments
Open
5 of 8 tasks

[UserStory] Add workarounds to Contributing.md #201

Aleksander-Gomez opened this issue Oct 4, 2023 · 8 comments
Labels
documentation Improvements or additions to documentation E: 1 Effort: 1 Story-Point S: Minor Severity: Minor - address if resource allow Edit user story User Story

Comments

@Aleksander-Gomez
Copy link

Aleksander-Gomez commented Oct 4, 2023

User Story

Essential components

  • Title describes the story
  • Stakeholder type is identified
  • Outcome is described
  • Rationale is explicit
  • Acceptance criteria are verifiable and from the perspective of the stakeholder

Story

As a contributor I want Workaround instructions for building on MacOS and Ubuntu machines in the contributing.md file
so that anyone who may be building on those machines can build successfully.

Acceptance Criteria

(Rules or scenarios are acceptable formats.)

  • * The contributing.md file offers a workaround for when building on MacOS and getting 'ant not recognized' build error
  • * The contributing.md file offers a workaround for when building on Ubuntu and building but jar files are failing to build
  • * Links to any mentioned 'technologies' home website for version and installation

Supporting Information

Related to epic #80, the contributing.md file could use a much more organized and chronological set of more specific instructions on what technologies are needed and how to build the project as well.

@Aleksander-Gomez Aleksander-Gomez added the user story User Story label Oct 4, 2023
@Aleksander-Gomez
Copy link
Author

I would like to be assigned this issue.

@jody
Copy link
Contributor

jody commented Oct 5, 2023

"Before you do anything..." seems like hyperbole. That also makes it hard to validate.

Questions about the "step by step instructions with terminal/command lines" include: "Which OS and user interface are being described?" and "How many specific contexts need to be addressed in the documentation?" It seems to me that a basic *nix OS and standard shell (bash, csh, zsh, ...) would be appropriate. We just need to make some decisions.

What is meant by "packages"?

@Aleksander-Gomez
Copy link
Author

After having talked about what we expect our audience to already know, Ive decided it better to suggest adding work-arounds to the contributing.md document for specific machines and problems that we've notice tend to arise when you try to follow the existing document.
More specifically, Id like to add two small work-arounds for users who try to build with a mac and run into a build error like 'ant isn't a recognized command', and for Ubuntu users who have a problem building the jar files. Two small additions to the document so that future users can build and avoid those build problems.

@jody
Copy link
Contributor

jody commented Oct 10, 2023

I agree. The approach you suggest is both highly valuable and feasible.

@jody
Copy link
Contributor

jody commented Oct 18, 2023

Please update the Acceptance Criteria of this issue to reflect your new intent. (Once we agree upon those, I will add this issue to the project.)

@Aleksander-Gomez Aleksander-Gomez changed the title [UserStory] Add clear build and 'pre-requisites' to Contributing.md [UserStory] Add workarounds to Contributing.md Oct 31, 2023
@Aleksander-Gomez Aleksander-Gomez removed their assignment Oct 31, 2023
@Aleksander-Gomez
Copy link
Author

We agreed on adding the 'workarounds' under the apache ant package header, as more of 'for your convenience' tips for installing ant using package managers for specific machines.

@jody jody added documentation Improvements or additions to documentation E: 1 Effort: 1 Story-Point S: Significant Severity: Significant - in the mainline plan labels Oct 31, 2023
Aleksander-Gomez pushed a commit to CS335001/redistricting that referenced this issue Nov 9, 2023
@Aleksander-Gomez
Copy link
Author

I'd like to suggest adding Junit as a tool in the contributing file. When you build this file, the build tool(ant) may be unable to locate the Junit library during the build, so users should make sure the JUnit JAR file is present in their project or is available in a directory that Ant can access.

@jody
Copy link
Contributor

jody commented Dec 8, 2023

I'd like to suggest adding Junit as a tool in the contributing file.

I agree! Contributors would benefit from more extensive information about development tools, especially those that perform quality assessment on contributed code.

@jody jody added S: Minor Severity: Minor - address if resource allow Edit and removed S: Significant Severity: Significant - in the mainline plan labels Dec 24, 2023
@jody jody moved this to To do in Redistricting a Region Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation E: 1 Effort: 1 Story-Point S: Minor Severity: Minor - address if resource allow Edit user story User Story
Projects
Development

No branches or pull requests

2 participants