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

Feedback on essay: Comparing DevOps Tools You Will Likely Encounter in the Workplace #1365

Merged
merged 3 commits into from
May 3, 2021

Conversation

fwallb
Copy link

@fwallb fwallb commented Apr 26, 2021

Feedback on essay

Member

Frida Wallberg - fwallb@kth.se - @fwallb

Feedback

This is an essay on an interesting subject that is very relevant to the course. Choosing the right tool that fits the needs of your project is important. The cost aspect is also essential since setting up a tool, and then having to choose another tool could imply loss of money. The essay has a great layout and figures that are visually pleasing. It is great that it is written in LaTex, which gives it a professional feel.

It is very ambitious to compare three types of software tools and three providers in each category. Doing this in less than 2000 words is a real challenge and gives approximately 200 words per provider. In your case with only ~1500, words due to many references, there are only 166 words per provider. This means that there is no space to go in depth to really understand their differences. Maybe it would have been better to focus on one tool, i.e CI tools, cloud providers or code repositories, to be able to describe them fully. What is the suggestion to improve and give the reader more information? Some of the facts in the text could be converted to a table or figure (like figure 2), to give more room for text. Maybe a few of the references could be removed to give more room for text?

The 2text-flow2 missing in some parts of the essay, and there are some grammar and spelling mistakes. A proofread and a run through Grammarly would improve the essay.

There are in some places, big jumps between sentences which makes the text hard to follow and understand. The language is somewhat informal and the tense used in some parts of the text is not fitting for a formal report. For example "This article will be looking at..." is future tense, but the article is finished, it would be better to use "This article looks at...".

Further, the use of first-person pronouns is advised against in formal writing. For example "...we will be comparing...", the sentence could be changed to _"Within continuous integration, Jenkins, Travis CI, and Circle CI highlighting tool effectiveness and use cases will be compared."

Some parts of the text, mainly the second section Different CI tools, consists of many very short sentences. Longer sentences are preferred in formal texts.

There are different usages of references throughout the text, especially when two references are used. You use [x][y], [x] [y], [x]. [y] and [x], [y] - would be better if it was consistent.

The title suggests that there is some comparison made in the essay, but it mainly consists of facts. It would have benefitted from a discussion at the end to really understand and get the author's perspective on the different tools.

There are many references, possibly too many, so that the substantial part of the article might be considered too short. 1558 words are included in the main part of the document and the rest are references. A large portion of the sentences are ended with a reference(s), which could give a messy feel.

1 Introduction

Good that the introduction is short and gives a good overview of what a reader can expect to learn from the essay.

Potential improvements

  • The last sentence is a bit bold - maybe change "gain confidence with such difficult decisions" to "have more information when making such decisions".

2 Different CI tools

  • I really like the figure! On a second look - it looks somewhat aggressive with Travis holding those sharp objects...
  • The aspects that are used for comparison are interesting and seem like good evaluation factors.

Potential improvements

  • In the first paragraph there is a "but" and then "only" which indicates some downgrading of the essay from the authors' point of view. Might be better to rephrase to something like "the three tools compared in this section are; Travis CI..."

  • The text in the third paragraph, starting with "Travis CI was known for...", feels like more of a list than a text when reading. The sentences do not work with each other. Maybe it is better to convert this part to a list of "Travis facts"? Or rewrite? After further thought, there might better to have a table with the different aspects, like:
    | | Jenkins | TravisCI | Circle CI |
    |---------- | --------|----------|-------------|
    |<aspect 1> | | | |
    |<aspect 2> | | | |

  • There is a gap between the forth and the fifth paragraph, seems like the paragraphs could be merged.

  • There is limited information about Travis CI, only 5 sentences, which is a little thin.

Grammar and spelling

  • One type of tool
  • There are many different software that provides
  • Open source software -> open-source software
  • As OSS... -> As an OSS...
  • ... customize their Jenkins setup to fit their needs.
  • Open source community -> open-source community
  • Code base -> codebase
  • ...using others requires tinkering

3 Different cloud providers

Good that there are a few introduction sentences in the beginning. Great that you have a figure that gives a good overview - more of those! It is easy to read and gives a lot of information quickly without using too many words.

Potential improvements

  • The second paragraph is only one sentence long, and could maybe be put together with the third paragraph.

Grammar and spelling

  • GCP’s cloud resources encompasses...
  • High profile -> high-profile
  • Pay-as-yo-go is spelled differently throughout
  • Cost effective -> cost-effective
  • google -> Google

4 Different code repositories

This part has a good introduction!

Potential improvements

  • The graph in figure 2 lacks y-axis label.
  • "...and Dow Jones are some of the customers that GitHub has..." could be rewritten as "...and Dow Jones are some of GitHub's customers." to improve clarity.
  • "...in terms of the number of users..."
  • The second to last and last paragraphs are separated but it seems like they could be merged.

Grammar and spelling

  • ...repository hosting company whose users
  • management oriented features -> management-oriented features
  • kanban -> Kanban

5 Conclusion

The conclusion is not really a conclusion, maybe it is more of a summary. Missing is some sort of ending and conclusion of the findings. This might give some inspiration.

References

The references are many, maybe too many? They seem to be correctly formated which is great!

@SophieHYe SophieHYe self-assigned this May 3, 2021
@SophieHYe
Copy link

Thanks for the feedback submission. I am now merging your PR.

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